[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