[PATCH] D27853: [CodeGenPrep]Restructure promoting Ext to form ExtLoad

[Jun Bum Lim via Phabricator via llvm-commits]

junbuml added a comment.

> You mean that you are going to add more transformation that will reuse that refactoring, right?

Yes, on top of this patch, I'm plaining on moving the transformation done in aarch64-type-promotion into CGP.  The main purpose of this patch is to prepare merging the transformation done in aarch64-type-promotion into CGP. And then, I want to implement https://reviews.llvm.org/D26524 in CGP.

> Second question, do you have performance number for the current version?

I haven't tried performance with/without the current version (moveExtToFormExtLoad()) itself.  I only ran spec2000/spec2006 in aarch64 for this change vs current version, and I didn't see any clear change in performance. I observed +3% in spec2006/astar only when performing the transformation tried in https://reviews.llvm.org/D26524, where I sunk SExt into its user blocks if foldable with its users. Please let me know if you have plan on current version itself.


https://reviews.llvm.org/D27853