[PATCH] D122616: RegAlloc: Fix verifier error after failed allocation
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 13 11:22:39 PDT 2022
arsenm added a comment.
In D122616#3449107 <https://reviews.llvm.org/D122616#3449107>, @qcolombet wrote:
> Hi @arsenm,
>
> The patch itself looks good but I wonder if we could maybe take a simpler road:
> Could we assign register zero (`noreg`) to this failed vreg?
>
> Cheers,
> -Quentin
The verifier doesn't like that at all since it doesn't have an associated class. Plus every target would have to handle encoding it as something in the end
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122616/new/
https://reviews.llvm.org/D122616
More information about the llvm-commits
mailing list