[llvm] [IndVars] Mark truncs as nuw/nsw (PR #88686)
Philip Reames via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 15 07:59:07 PDT 2024
https://github.com/preames approved this pull request.
LGTM
One of the thoughts I'd had when seeing this langref proposal was that it might allow us to simplify this code significantly. I believe one of the things which makes this code complex is that we have to "commit" to our signed or unsigned interpretation by performing all rewrites at once since we couldn't materialize a wide IV + truncate and preserve the information about the choice. With this IR change, we now have that possibility. This might not fully work out - there's other reasons for complexity in this code - but mentioning it in case it inspires you. :)
https://github.com/llvm/llvm-project/pull/88686
More information about the llvm-commits
mailing list