[llvm] [IR] Add nowrap flags for trunc instruction (PR #85592)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 18 08:59:51 PDT 2024


nikic wrote:

> > Another place to update is SCEVExpander, which backs up poison-generating flags. Needs an adjustment to handle these on trunc now. (Don't think this needs a test, it's pretty tricky to do that.)
> 
> before modifying the SCEVExpander, are these places legal to modify [here](https://github.com/llvm-mirror/llvm/blob/master/lib/Analysis/ScalarEvolution.cpp#L4434), should we add a class like [this](https://github.com/llvm-mirror/llvm/blob/master/lib/Analysis/ScalarEvolution.cpp#L4470), should we modift [this](https://llvm.org/doxygen/ScalarEvolution_8cpp_source.html#l02382) to

You should not modify any of those places (at least in this PR), only https://github.com/llvm/llvm-project/blob/12b802ac0bc6ddf0742aa3fe8caecd8204d70ca5/llvm/lib/Transforms/Utils/ScalarEvolutionExpander.cpp#L46-L75

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


More information about the llvm-commits mailing list