[PATCH] Implement more expansions of extloads
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