Proof of concept patch. I found this while analyzing some of the regressions in http://reviews.llvm.org/D19401 which are fixed by this, but the change should be good in general.
No testcase updates included here yet.
Differential D21540
AMDGPU: Define a schedule class for COPY. MatzeB on Jun 20 2016, 4:47 PM. Authored by
Details Proof of concept patch. I found this while analyzing some of the regressions in http://reviews.llvm.org/D19401 which are fixed by this, but the change should be good in general. No testcase updates included here yet.
Diff Detail
Event Timeline
Comment Actions Slightly enhanced schedule class selection for COPY. This really should be enough to get http://reviews.llvm.org/D19401 committed. |
copy should be either WriteSALU or WriteVALU, but both of these seem to just use SALU. Is there a way to change this depending on the instance of the instruction?