[PATCH] R600: Make sign_extend_inreg legal.
Matt Arsenault
arsenm2 at gmail.com
Mon Apr 21 09:54:04 PDT 2014
On Apr 21, 2014, at 6:40 AM, Tom Stellard <tom at stellard.net> wrote:
> On Thu, Apr 17, 2014 at 10:37:12PM -0700, Matt Arsenault wrote:
>>
>> On Apr 2, 2014, at 9:59 AM, Tom Stellard <tom at stellard.net> wrote:
>>
>>> I want to move away from using patterns with INSERT_SUBREG, because
>>> it just adds another case we need to handle in the SIFixSGPRCopies pass.
>>> Can you move this pattern into AMDILISelDAGToDAG.cpp and use REG_SEQUENCE
>>> instead of INSERT_SUBREG + IMPLICIT_DEF.
>>
>> Now that it handles INSERT_SUBREG anyway, should I go back to using the patterns?
>>
>
> What are the advantages of using the patterns?
Less code
More information about the llvm-commits
mailing list