Skip to content

Commit

Permalink
Fix-forward 'RegAllocFast: Avoid using temporary DiagnosticInfo llvm#…
Browse files Browse the repository at this point in the history
…120184' (llvm#120268)

There was a buildbot breakage

(https://lab.llvm.org/buildbot/#/builders/24/builds/3329/steps/11/logs/stdio):


/home/b/sanitizer-aarch64-linux-bootstrap-asan/build/llvm-project/llvm/test/CodeGen/AMDGPU/ran-out-of-registers-error-all-regs-reserved.ll:9:10:
error: CHECK: expected string not found in input
; CHECK: error: <unknown>:0:0: no registers from class available to
allocate in function 'no_registers_from_class_available_to_allocate'

2: ==75198==ERROR: AddressSanitizer: stack-use-after-scope on address
0xfa23f9f1c270 at pc 0xb2660dda9340 bp 0xfffffe8ab340 sp 0xfffffe8ab338

caused by llvm#120184, which made a
partial fix but also renabled the tests. This patch attempts to fix
forward by applying the same fix to the error message highlighted in the
buildbot.
  • Loading branch information
thurstond authored Dec 17, 2024
1 parent 30916b6 commit e8a6563
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions llvm/lib/CodeGen/RegAllocFast.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1187,10 +1187,9 @@ MCPhysReg RegAllocFastImpl::getErrorAssignment(const LiveReg &LR,
if (AllocationOrder.empty()) {
const Function &Fn = MF.getFunction();
if (EmitError) {
DiagnosticInfoRegAllocFailure DI(
Fn.getContext().diagnose(DiagnosticInfoRegAllocFailure(
"no registers from class available to allocate", Fn,
MI.getDebugLoc());
Fn.getContext().diagnose(DI);
MI.getDebugLoc()));
}

ArrayRef<MCPhysReg> RawRegs = RC.getRegisters();
Expand Down

0 comments on commit e8a6563

Please sign in to comment.