[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