[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:14:29 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