[PATCH] D26648: Clarify semantic of reserved registers.
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 14 16:13:34 PST 2016
arsenm added a comment.
In https://reviews.llvm.org/D26648#595163, @MatzeB wrote:
> This is an early patch to get the discussion going. Some things are still missing:
>
> - A bunch of targets violate the superreg rule (e.g. AArch64 reserving XZR but not tuples like %XZR_X0). AArch64, ARM, Hexagon, Mips and PowerPC currently fail the verifier. So far it looks like actual target bugs to me.
> - I think we should add this rule: "If a reserved subregister has subregister at least one subregister (or register unit) must be reserved", yes?
AMDGPU has hit bugs from only reserving the super register before, and not the subregisters
Repository:
rL LLVM
https://reviews.llvm.org/D26648
More information about the llvm-commits
mailing list