[PATCH] D65967: [SeparateConstOffsetFromGEP][PowerPC] Fix: sext(a) + sext(b) -> sext(a + b) matches add and sub instructions with one another

Andrew J Wock via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 1 11:27:31 PDT 2019


ajwock updated this revision to Diff 227494.
ajwock added a comment.

Sorry for the delay- I had to investigate some of the fragility so that my test case reduced to this pass would still trigger the issue.  However, I did manage to get a test case that is only a few lines long that would trigger the issue without the update.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65967

Files:
  lib/Transforms/Scalar/SeparateConstOffsetFromGEP.cpp
  test/Transforms/SeparateConstOffsetFromGEP/PowerPC/test-add-sub-separation.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D65967.227494.patch
Type: text/x-patch
Size: 5526 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191101/01001ed2/attachment.bin>


More information about the llvm-commits mailing list