[PATCH] D132022: [SCEVExpander] Recompute poison-generating flags on hoisting. PR57187

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 19 01:14:28 PDT 2022


mkazantsev added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/ScalarEvolutionExpander.cpp:1065
+            ScalarEvolution::maskFlags(*Flags, SCEV::FlagNSW) == SCEV::FlagNSW);
+      }
   }
----------------
This piece of code replicates similar piece of code in IndVars, I propose refectoring as a separate patch. Now we just need a conceptual agreement on how we fix it.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132022/new/

https://reviews.llvm.org/D132022



More information about the llvm-commits mailing list