[PATCH] D30416: [InstCombine] Redo reduceLoadOpStoreWidth in instcombine for bitfield store optimization.

Wei Mi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 4 12:46:44 PST 2017


wmi updated this revision to Diff 90589.
wmi added a reviewer: eli.friedman.
wmi added a comment.

Update patch according to Eli's comments.

- Add safety check to ensure no memory modifying inst bewteen load and store.
- Extend the shrinking funcationality to cover the cases Eli gave to me.
- Code refactoring so that different shrinking requirements can share the code as much as they can.


Repository:
  rL LLVM

https://reviews.llvm.org/D30416

Files:
  include/llvm/IR/PatternMatch.h
  lib/CodeGen/CodeGenPrepare.cpp
  test/CodeGen/ARM/bitfield-store.ll
  test/CodeGen/ARM/illegal-bitfield-loadstore.ll
  test/CodeGen/X86/bitfield-store.ll
  test/CodeGen/X86/illegal-bitfield-loadstore.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30416.90589.patch
Type: text/x-patch
Size: 51232 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170304/afb2d610/attachment.bin>


More information about the llvm-commits mailing list