[llvm] [polly] [SCEV] Rename SCEVNoWrapFlags to SCEVFlags (NFC) (PR #225179)

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 29 01:55:01 PDT 2026


================
@@ -997,9 +1007,8 @@ class SCEVLoopAddRecRewriter
 };
 
 template <typename SCEVPtrT>
-inline SCEVUseT<SCEVPtrT>::SCEVUseT(SCEVPtrT S, SCEVNoWrapFlags Flags)
-    : Base(S, 0) {
-  if (any(Flags)) {
+inline SCEVUseT<SCEVPtrT>::SCEVUseT(SCEVPtrT S, SCEVFlags Flags) : Base(S, 0) {
+  if (any(Flags & SCEVFlags::FlagsNoWrapMask)) {
----------------
fhahn wrote:

Should this instead drop flags outside `SCEVFlags::FlagsNoWrapMask`? Otherwise , if we add other flags, then we could set unexpected flags below?

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


More information about the llvm-commits mailing list