[PATCH] D91562: [LSR] Drop poison flags for post-inc IVs (PR46943)
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 17 11:53:20 PST 2020
nikic added a comment.
The approach here doesn't look right to me. Poison flags have no relation to the position where a certain instruction is executed. What's relevant is where the instruction is used.
This transform adds a new poison-is-UB user to the post-inc IV. Whereever that code is, should be the place that drops poison flags. The code is pretty convoluted though, so I didn't immediately spot where that happens.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91562/new/
https://reviews.llvm.org/D91562
More information about the llvm-commits
mailing list