[PATCH] D20591: [InstCombine] Catch more bswap cases missed due to zext and truncs.
Geoff Berry via llvm-commits
llvm-commits at lists.llvm.org
Wed May 25 10:40:46 PDT 2016
gberry added inline comments.
================
Comment at: test/Transforms/InstCombine/bswap.ll:106
@@ +105,3 @@
+ %conv = zext i16 %a to i32
+ %shr = lshr i16 %a, 8
+ %shl = shl i32 %conv, 8
----------------
Shouldn't this be an i32 lshr of %conv and the second zext below deleted if you want the IR to look like it would for a target without i16 as a legal type?
http://reviews.llvm.org/D20591
More information about the llvm-commits
mailing list