[PATCH] D17032: [X86] Add a pass to change byte and word instructions to zero-extending versions.

Kevin B. Smith via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 10 16:05:56 PST 2016


kbsmith1 added a comment.

In http://reviews.llvm.org/D17032#349278, @echristo wrote:

> Naming is hard and shouldn't block anything here. That said, what do you think about using this to get rid of some of the reg stall code that was listed? :)


I am in favor of it.  I think the staging should be:

- Get this in.
- Get it turned on by default.
- Pull the other changes out, 1 by 1, making sure that this catches those cases, improving the safety analysis, or heuristics as necessary to make sure that performance is maintained/improved.

Each of those stages seems like it ought to be a separate change, following on to this.


http://reviews.llvm.org/D17032





More information about the llvm-commits mailing list