[PATCH] D125653: [TypePromotion] Fix another case for sext vs zext in promoted constant.

[Dave Green via Phabricator via llvm-commits]

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