[PATCH] D15230: [BranchFolding] Merge MMOs during tail merge

Junmo Park via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 4 06:50:04 PST 2015


flyingforyou created this revision.
flyingforyou added reviewers: gberry, mcrosier.
flyingforyou added subscribers: rafael, llvm-commits.

Recommit r254694, it was reverted in r254700 to investigate a bot failure.

If we remove the MMOs from Load/Store instructions,
they are treated as volatile. This makes other optimization passes unhappy.
eg. Load/Store Optimization

So, it looks better to merge, not remove.
But if the product of two MMOs count is higher than threshold,
we just remove them for preventing any pathological cases from
causing compile time blowups or overflow.

http://reviews.llvm.org/D15230

Files:
  lib/CodeGen/BranchFolding.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15230.41874.patch
Type: text/x-patch
Size: 4176 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151204/c9c956fe/attachment.bin>


More information about the llvm-commits mailing list