[PATCH] D125653: [TypePromotion] Fix another case for sext vs zext in promoted constant.
Dave Green via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 19 01:29:26 PDT 2022
dmgreen added a comment.
If I haven't got this wrong, this says we could just always be zexting the constant from a sub:
https://alive2.llvm.org/ce/z/kTAyt2
https://alive2.llvm.org/ce/z/a6HHq4
That sounds like it would make simpler constants too, although I'm not sure there are any tests that show it.
What do you think? Otherwise I believe this is correct.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125653/new/
https://reviews.llvm.org/D125653
More information about the llvm-commits
mailing list