[PATCH] D34368: [AArch64] PointerRegClass should be GPR64spRegClass
Kristof Beyls via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 23 07:43:13 PDT 2017
kristof.beyls added a comment.
In https://reviews.llvm.org/D34368#784700, @kongyi wrote:
> In https://reviews.llvm.org/D34368#784693, @efriedma wrote:
>
> > Testcase? (I think this affects inline asm.)
>
>
> This change is actually laying ground for fixing an inline asm bug (PR33134). Test cases will be added along with rest of the fix in a follow up change.
This seems to be a fix for a correctness bug. If you do have a follow-up change that will have test cases that demonstrate this fixes things, maybe it's best to combine this change with the follow up change, especially since this patch is a single-line change?
https://reviews.llvm.org/D34368
More information about the llvm-commits
mailing list