[PATCH] D110246: [InstSimplify][InstCombine] Fold ptrtoint(gep i8 null, x) -> x
Alexander Richardson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 24 01:00:31 PDT 2021
arichardson added a comment.
In D110246#3018908 <https://reviews.llvm.org/D110246#3018908>, @nikic wrote:
> I'm not convinced that it makes sense to add this to InstSimplify. This is extra complexity to cover a really narrow subset of what InstCombine will handle anyway. This already seems like quite the edge case fold -- is there some reason to believe that it is critical to handle the "i8 with variable index" special case in InstSimplify?
I think you are right, for us it seems like the most important cases will be handled by constant folding and deferring the rest to InstCombine instead of InstSimplify should address the rest. Will drop this and just focus on the InstCombine fold.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110246/new/
https://reviews.llvm.org/D110246
More information about the llvm-commits
mailing list