This is an archive of the discontinued LLVM Phabricator instance.

Compiler-rt changes to react to skiping promotable allocas.
AbandonedPublic

Authored by zaks.anna on Feb 18 2015, 5:26 PM.

Details

Reviewers
kcc
samsonov
Summary

This is the compiler-rt part of the following change: "Skip promotable allocas to improve performance at -O0" http://reviews.llvm.org/D7741.

Diff Detail

Event Timeline

zaks.anna updated this revision to Diff 20241.Feb 18 2015, 5:26 PM
zaks.anna retitled this revision from to Compiler-rt changes to react to skiping promotable allocas..
zaks.anna updated this object.
zaks.anna edited the test plan for this revision. (Show Details)
zaks.anna added reviewers: kcc, samsonov.
zaks.anna added subscribers: Unknown Object (MLST), kubamracek.

I've submitted http://reviews.llvm.org/D7809 and http://reviews.llvm.org/D7810 which properly fixes these tests (the failures are not related to the "promotable allocas" patch, but they fail on higher -O levels as well).

samsonov edited edge metadata.Feb 21 2015, 3:47 PM

Thanks for looking into this, Kuba!

I believe this is now unnecessary, as both http://reviews.llvm.org/D7809 and http://reviews.llvm.org/D7810 have landed.

zaks.anna abandoned this revision.Jul 23 2015, 5:33 PM