[all-commits] [llvm/llvm-project] 33f843: RegAlloc: Use DiagnosticInfo to report register al...

[Matt Arsenault via All-commits]

  Branch: refs/heads/users/arsenm/regalloc-use-diagnostic-info-on-failure
  Home:   https://github.com/llvm/llvm-project
  Commit: 33f843993aec87c2ea371d1213c3145c7e61f63f
      https://github.com/llvm/llvm-project/commit/33f843993aec87c2ea371d1213c3145c7e61f63f
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2024-12-16 (Mon, 16 Dec 2024)

  Changed paths:
    M llvm/include/llvm/IR/DiagnosticInfo.h
    M llvm/lib/CodeGen/RegAllocBase.cpp
    M llvm/lib/CodeGen/RegAllocFast.cpp
    M llvm/lib/IR/DiagnosticInfo.cpp
    M llvm/test/CodeGen/AArch64/arm64-anyregcc-crash.ll
    M llvm/test/CodeGen/AMDGPU/illegal-eviction-assert.mir
    M llvm/test/CodeGen/AMDGPU/issue48473.mir
    A llvm/test/CodeGen/AMDGPU/ran-out-of-registers-error-all-regs-reserved.ll
    A llvm/test/CodeGen/AMDGPU/ran-out-of-registers-errors.ll
    M llvm/test/CodeGen/AMDGPU/remaining-virtual-register-operands.ll
    M llvm/test/CodeGen/PowerPC/ppc64-anyregcc-crash.ll
    M llvm/test/CodeGen/X86/anyregcc-crash.ll

  Log Message:
  -----------
  RegAlloc: Use DiagnosticInfo to report register allocation failures

Improve the non-fatal cases to use DiagnosticInfo, which will now
provide a location. The allocators attempt to report different errors
if it happens to see inline assembly is involved (this detection is
quite unreliable) using srcloc instead of dbgloc. For now, leave this
behavior unchanged. I think reporting the full location and context
function would be more useful.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications