OSDN Git Service

[Sema] Try 2: Sanity-check alignment requested via `__attribute__((assume_aligned...
authorRoman Lebedev <lebedev.ri@gmail.com>
Thu, 23 Jan 2020 19:48:57 +0000 (22:48 +0300)
committerRoman Lebedev <lebedev.ri@gmail.com>
Fri, 24 Jan 2020 13:56:39 +0000 (16:56 +0300)
commit0a002f679be4a9d363e6f21b707493c18158930b
tree94643af1ea9ffe5fd8e22006ab755639dd9840f1
parent5cee34013cfd4edea0041bc0b73121cc4ad76ccf
[Sema] Try 2: Sanity-check alignment requested via `__attribute__((assume_aligned(imm)))`

Summary:
For `__builtin_assume_aligned()`, we do validate that the alignment
is not greater than `536870912` (D68824), but we don't do that for
`__attribute__((assume_aligned(N)))` attribute.
I suspect we should.

This was initially committed in a4cfb15d15a8a353fe316f2a9fe1c8c6a6740ef1
but reverted in 210f0882c9e5d6f504b8f29e8a5eae884f812e5c due to
suspicious bot failures.

Reviewers: erichkeane, aaron.ballman, hfinkel, rsmith, jdoerfert

Reviewed By: erichkeane

Subscribers: cfe-commits, llvm-commits

Tags: #llvm, #clang

Differential Revision: https://reviews.llvm.org/D72994
clang/lib/Sema/SemaDeclAttr.cpp
clang/test/Sema/builtin-assume-aligned.c