[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