[PATCH] D122616: RegAlloc: Fix verifier error after failed allocation

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 13 11:21:45 PDT 2022

Previous message: [PATCH] D123699: [AMDGPU] Remove redundand RequiredAlignment assignment. NFCI.
Next message: [PATCH] D122616: RegAlloc: Fix verifier error after failed allocation
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

qcolombet added a comment.

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


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122616/new/

https://reviews.llvm.org/D122616

Previous message: [PATCH] D123699: [AMDGPU] Remove redundand RequiredAlignment assignment. NFCI.
Next message: [PATCH] D122616: RegAlloc: Fix verifier error after failed allocation
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list