[PATCH] D56156: [DAGCombiner][X86][PowerPC] Teach visitSIGN_EXTEND_INREG to fold (sext_in_reg (aext/sext x)) -> (sext x) when x has more than 1 sign bit and the sext_inreg is from one of them.

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 30 07:32:35 PST 2018


hfinkel added inline comments.


================
Comment at: test/CodeGen/PowerPC/f128-truncateNconv.ll:358
 ; CHECK-NEXT:    mfvsrwz r3, v2
-; CHECK-NEXT:    extsh r3, r3
+; CHECK-NEXT:    extsw r3, r3
 ; CHECK-NEXT:    blr
----------------
This test-case change looks technically correct (note that this will change behavior of programs with UB (if the value being converted to an integer wasn't really representable in the smaller integer size)).


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56156/new/

https://reviews.llvm.org/D56156





More information about the llvm-commits mailing list