[PATCH] D42783: [DAGCombiner] When folding fold (sext/zext (and/or/xor (sextload/zextload x), cst)) -> (and/or/xor (sextload/zextload x), (sext/zext cst)) make sure we check the legality of the full extended load.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 3 12:20:13 PST 2018


craig.topper added a comment.

The other consequence of that is that any additional users of the load aren't checked for the isTruncateFree in ExtendUsesToFormExtLoad. So the test case I regressed in https://reviews.llvm.org/D42679 only works today by accident because we didn't check the additional load user which was a truncate.


Repository:
  rL LLVM

https://reviews.llvm.org/D42783





More information about the llvm-commits mailing list