[PATCH] D50931: Consider isLegalAddressingImm in Constant Hoisting

Z. Zheng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 27 16:08:03 PDT 2018


zzheng updated this revision to Diff 162767.
zzheng marked an inline comment as done.
zzheng added a comment.

Previous version mistakenly checks MinValItr->ConstInst rather than CC->ConstInst against StoreInst User's pointer operand.


Repository:
  rL LLVM

https://reviews.llvm.org/D50931

Files:
  lib/Transforms/Scalar/ConstantHoisting.cpp
  test/Transforms/ConstantHoisting/ARM/is-legal-addressing-imm.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50931.162767.patch
Type: text/x-patch
Size: 6775 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180827/7c77d2f2/attachment.bin>


More information about the llvm-commits mailing list