[PATCH] Tweak EarlyCSE to recognize series of dead stores

Philip Reames listmail at philipreames.com
Mon Nov 17 17:39:04 PST 2014


Hi hfinkel, nlewycky, craig.topper,

EarlyCSE is giving up on the current instruction immediately when it recognizes that the current instruction makes a previous store trivially dead.  There's no reason to do this.  Once the previous store has been deleted, it's perfectly legal to remember the value of the current store (for value forwarding) and the fact the store occurred (it could be dead too!).

http://reviews.llvm.org/D6301

Files:
  lib/Transforms/Scalar/EarlyCSE.cpp
  test/Transforms/EarlyCSE/basic.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6301.16311.patch
Type: text/x-patch
Size: 1821 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141118/c0a342dd/attachment.bin>


More information about the llvm-commits mailing list