[Patch] Enable value forwarding for loads from calloc without intervening store

Duncan P. N. Exon Smith dexonsmith at apple.com
Thu Jun 12 08:16:17 PDT 2014


> On 2014 Jun 11, at 10:15, Philip Reames <listmail at philipreames.com> wrote:
> 
> Attached.

Thanks.

> 
> http://reviews.llvm.org/D3931
> 
> Philip
> <calloc-load-forward.diff>

LGTM.  Do you need me to commit it for you?

BTW, I noticed another strange whitespace thing -- if I'm committing it,
I'll just fix this myself, but if you're committing it please run
clang-format:

+      ValuesPerBlock.push_back(AvailableValueInBlock::get(DepBB,
+                                             Constant::getNullValue(LI->getType())));

This indentation is strange (and violates 80-columns).  clang-format's
opinion is this:

      ValuesPerBlock.push_back(AvailableValueInBlock::get(
          DepBB, Constant::getNullValue(LI->getType())));




More information about the llvm-commits mailing list