[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