[clang-tools-extra] [llvm] [compiler-rt] [clang] [flang] [clang] Add support for new loop attribute [[clang::code_align()]] (PR #70762)

[Erich Keane via cfe-commits]

================
@@ -334,11 +334,12 @@ CodeAlignAttr *Sema::BuildCodeAlignAttr(const 
AttributeCommonInfo &CI,
     // This attribute requires an integer argument which is a constant power of
     // two between 1 and 4096 inclusive.
     int AlignValue = ArgVal.getSExtValue();
-    if (AlignValue < CodeAlignAttr::getMinValue() ||
-        AlignValue > CodeAlignAttr::getMaxValue() || !ArgVal.isPowerOf2()) {
+    static int MaximumAlignment = 4096;
----------------
erichkeane wrote:

Ah,  I meant these still as addition members, as static-constexpr.  That way it 
is still `CodeAlignAttr::MinimumAlignment` and 
`CodeAlignAttr::MaximumAlignment`, but not as a function that implies that it 
is not a constant value.

https://github.com/llvm/llvm-project/pull/70762
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits