[llvm] [polly] [SCEV] Refactor SCEV flags, in preparation to extend (NFC) (PR #225179)

David Sherwood via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 22 05:45:14 PDT 2026


================
@@ -107,34 +107,34 @@ LLVM_ABI extern bool VerifySCEV;
 /// can trap) can be defined per these rules in regions where it would trap
 /// at runtime.  A SCEV being defined does not require the existence of any
 /// instruction within the defined scope.
-enum class SCEVNoWrapFlags {
-  FlagAnyWrap = 0,    // No guarantee.
+enum class SCEVFlags {
+  FlagAny = 0,        // No guarantee.
----------------
david-arm wrote:

Given that the goal is to have more than just wrapping flags here, is it also worth renaming `FlagAny` to `FlagNone`? Once the exact flag is added it's a bit ambiguous what `FlagAny` means I think.

https://github.com/llvm/llvm-project/pull/225179


More information about the llvm-commits mailing list