Skip to content

Commit 210f088

Browse files
committed
Revert "[Sema] Sanity-check alignment requested via __attribute__((assume_aligned(imm)))"
Likely makes bots angry. This reverts commit a4cfb15. # Conflicts: # clang/lib/Sema/SemaDeclAttr.cpp
1 parent 00756b1 commit 210f088

File tree

2 files changed

+0
-5
lines changed

2 files changed

+0
-5
lines changed

clang/lib/Sema/SemaDeclAttr.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1625,10 +1625,6 @@ void Sema::AddAssumeAlignedAttr(Decl *D, const AttributeCommonInfo &CI, Expr *E,
16251625
<< E->getSourceRange();
16261626
return;
16271627
}
1628-
1629-
if (I > Sema::MaximumAlignment)
1630-
Diag(CI.getLoc(), diag::warn_assume_aligned_too_great)
1631-
<< CI.getRange() << Sema::MaximumAlignment;
16321628
}
16331629

16341630
if (OE) {

clang/test/Sema/builtin-assume-aligned.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@ int test8(int *a, int j) {
4646
void test_void_assume_aligned(void) __attribute__((assume_aligned(32))); // expected-warning {{'assume_aligned' attribute only applies to return values that are pointers}}
4747
int test_int_assume_aligned(void) __attribute__((assume_aligned(16))); // expected-warning {{'assume_aligned' attribute only applies to return values that are pointers}}
4848
void *test_ptr_assume_aligned(void) __attribute__((assume_aligned(64))); // no-warning
49-
void *test_ptr_assume_aligned(void) __attribute__((assume_aligned(1073741824))); // expected-warning {{requested alignment must be 536870912 bytes or smaller; maximum alignment assumed}}
5049

5150
int j __attribute__((assume_aligned(8))); // expected-warning {{'assume_aligned' attribute only applies to Objective-C methods and functions}}
5251
void *test_no_fn_proto() __attribute__((assume_aligned(32))); // no-warning

0 commit comments

Comments
 (0)