[PATCH] D49512: [CodeGenPrepare] Add BothExtension type to PromotedInsts

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 6 17:15:35 PDT 2018


qcolombet added a comment.

To be more precise, the latest promotion should be correct, but if we want the full information of multiple extension, the current patch is I believe wrong.
E.g., zext (sext i8 to i16) to i32, doesn't give you the same warranties on the bit pattern than what zext i8 to i32 gives, whereas we won't be able to see that.


https://reviews.llvm.org/D49512





More information about the llvm-commits mailing list