- User Since
- Nov 17 2016, 12:59 PM (117 w, 6 d)
Mon, Feb 18
Adding support for scalarizing was 2 LOC, so meh, let's just do it here.
Since it doesn't look like there's really any way to unify these right now, I think this LGTM.
Wed, Feb 13
Oh no! LGTM
LGTM with question
Tue, Feb 12
Hmm, It might be nice to have this as a specialization of pointer_iterator if possible.
Updating diff to address comments on test before committing
Mon, Feb 11
Rebased after removing redundant IR from existing .mir test
Updated existing .mir test instead of adding a .ll test
Fri, Feb 8
Relanded in r353589. This works fine after r353586.
I think that this is safe to recommit after the changes in https://reviews.llvm.org/D57980.
Wed, Feb 6
Update diff to address nit before committing
Tue, Feb 5
Use a function containing a switch instead of a map, and move it all into IRTranslator.cpp.
Mon, Feb 4
This seems like a pretty good idea... LGTM, aside from the one nit.
@arsenm, adding this opcode breaks AMDGPU somehow. Do you have any idea why that might be?
Wed, Jan 30
Committed in r352692
Committed in r352684
Seems reasonable, LGTM
Committed in r352673
Thanks! Committed in r352671.
Thanks! Committed in r352670.