[PATCH] D56175: [PowerPC] Exploit store instructions that store a single vector element

[Nemanja Ivanovic via Phabricator via llvm-commits]

nemanjai added a comment.

In D56175#1356963 <https://reviews.llvm.org/D56175#1356963>, @jsji wrote:

> LGTM, as long as we add the missing BE test. Thanks for exploiting this, and also great patience while discussion.


Not at all, thank you for the close scrutiny of the changes. I always feel much more at ease committing something after a thorough discussion in the review than something that just gets an automatic approval. Also yup, I'll add the BE test on the commit.

Also, some SPEC numbers with this transformation for motivation:
blender - 6.5%/9.5% improvement on base/peak rate
gcc - 2% improvement on both base/peak rate
deepsjeng - 1.31% improvement on base rate
xz - 15.9% improvement on base

lbm - 1.38% degradation on peak rate, no observable change in base rate

All other changes are below 1% with the majority of those in the improvement category.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56175/new/

https://reviews.llvm.org/D56175