[PATCH] Implement more expansions of extloads

Matt Arsenault Matthew.Arsenault at amd.com
Tue Jan 13 16:26:21 PST 2015


On 01/13/2015 04:24 PM, Matt Arsenault wrote:
> On 01/13/2015 04:03 PM, Ahmed Bougacha wrote:
>> This contradicts the outer if, no? Should it be just 
>> "TLI.isTypeLegal(SrcVT)" ?
>> If so, since this doesn't break, I'm guessing it's no longer tested?
>> (not that I know where one would do that.)
> Yes, that block is dead code. I'm not sure how that's still there, all 
> of my tests pass with it deleted.
Actually, it isn't dead but should be implementing what was there 
before. Even with it deleted, it passes so evidently the existing 
handling isn't well tested



More information about the llvm-commits mailing list