[PATCH] D34183: AMDPU: Cleanup CreateLiveInRegister
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 19 13:49:18 PDT 2017
> On Jun 19, 2017, at 13:30, Tom Stellard via llvm-commits <llvm-commits at lists.llvm.org> wrote:
>
> On 06/19/2017 04:28 PM, Matt Arsenault via Phabricator wrote:
>> arsenm added a comment.
>>
>> This breaks an r600 test. Why does the SI version add the wrapper CopyFromReg and not r600?
>>
>
> I'm not sure.
>
I think it’s necessary because then it creates the illegal SGPR->VGPR copy SIFixSGPRCopies looks for. That could probably be fixed by looking at users of the input virtual registers though.
More information about the llvm-commits
mailing list