[PATCH] D12588: PeepholeOptimizer: Look through simple extracts of reg_sequence

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 3 09:27:55 PDT 2015


qcolombet added a subscriber: qcolombet.
qcolombet requested changes to this revision.
qcolombet added a reviewer: qcolombet.
qcolombet added a comment.
This revision now requires changes to proceed.

Hi Matt,

The changes in the peephole optimizer and AMDGPU seem to be to separate patches to me.
Please stage them like that if this is indeed the case.

Also, the thing that you add to the rewriter does not fit with the intended design, see inline comments.

Cheers,
-Quentin


================
Comment at: lib/CodeGen/PeepholeOptimizer.cpp:1252
@@ +1251,3 @@
+  //
+  //  => vreg3 = COPY vreg0
+  //
----------------
The design of the copy rewriter is to find alternative sources within the value tracker.
This change does not fit that scheme and looks like an ad hoc solution.
I believe what it is missing is to teach the value tracker about sub register.
Could you look into that direction or explain why do you think we should do it like you propose in this patch?


http://reviews.llvm.org/D12588





More information about the llvm-commits mailing list