Wrong source range for local/global declarations with unused/alias attribute.

[hokein]

See https://godbolt.org/z/bG5cEzY4Y

struct TestType {
[[maybe_unused]] int unsafe_one[16]; // #1 good, it is fixed by https://github.com/llvm/llvm-project/commit/b3a6d434a7051d879718ef92a4fafd1697759aed
};

[[maybe_unused]] int unsafe_one[16]; // #2 bad

void func() {
[[maybe_unused]] int unsafe_one[16]; // #3 bad
}

The source range of the declaration should include the unused attribute.

[llvmbot]

@llvm/issue-subscribers-clang-frontend

Author: Haojian Wu (hokein)

See https://godbolt.org/z/bG5cEzY4Y

struct TestType {
[[maybe_unused]] int unsafe_one[16]; // #<!-- -->1 good, it is fixed by https://github.com/llvm/llvm-project/commit/b3a6d434a7051d879718ef92a4fafd1697759aed
};

[[maybe_unused]] int unsafe_one[16]; // #<!-- -->2 bad

void func() {
[[maybe_unused]] int unsafe_one[16]; // #<!-- -->3 bad
}

The source range of the declaration should include the unused attribute.