- User Since
- Dec 17 2012, 10:03 AM (348 w, 1 d)
Fri, Aug 16
Tue, Aug 13
LGTM with a test case.
LGTM for the generic part.
Thu, Aug 1
Jul 19 2019
Jul 8 2019
It was added by D46315
Remove loop entirely. Ultimately determineCalleeSaves adds everything from MCRegAliasIterator anyway
Jul 5 2019
Let's go with that for now with one caveat. Does the code build without ANDGPU?
I.e., does the callingconv enum get generated?
Jul 3 2019
Jul 2 2019
General direction LGTM but there's a bug in the current patch unless I am mistaken.
Jun 19 2019
I second Daniel's comment, this is RegBankSelect's job to do this choice and the Legalizer shouldn't need to know about f64 vs. s64.
Maybe adding an MCRegister that's nearly identical might be a future improvement? That way it could enforce MCRegister->Register is OK, but Register->MCRegister is not
Jun 18 2019
Thanks for doing this Matt, this is a long due clean-up.
I haven't looked carefully at all the changes, but the general direction LGTM.
Jun 17 2019
Those rules work when there's a load/store for every type but they run into unable-to-legalize if that's not the case.
Jun 14 2019
LGTM modulo want @paquette pointed out.
Jun 13 2019
Jun 12 2019
Agree, the fix is more important than the performance loss at this point.
Jun 5 2019
Jun 3 2019
May 30 2019
What is the use case for this?
May 28 2019
May 27 2019
Nitpicks on the test below.
May 24 2019
May 16 2019
Please provide us with more details about your idea.
May 14 2019
May 13 2019
did anything like this come up in the past?
Looks good to me, modulo the order in which we check the hints.
May 8 2019
May 3 2019
@Quentin: This has the same common-code change as in D58923, with the added VRM to foldMemoryOperand(). You seemed fine with this change, right?
LGTM. Nitpicks below.
May 2 2019
Apr 29 2019
Apr 22 2019
Apr 18 2019
Could you please update the summary before pushing this?
Apr 16 2019
- Use comma separated list instead of group naming.
Apr 15 2019
The source change looks good to me, but I would like a bit more work on the test itself.
Apr 4 2019
Looks reasonable to me.
Apr 3 2019
Are the dead instructions marked during the detect dead lanes pass or during the rename independent SubReg pass?
The generic changes look sensible to me.
I would just suggest to add a comment on what VRM is for on the modified methods.