[all-commits] [llvm/llvm-project] 204d8a: 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: 204d8aaacd2356f4c67a1b006cc94f283bf42b80
      https://github.com/llvm/llvm-project/commit/204d8aaacd2356f4c67a1b006cc94f283bf42b80
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2024-12-11 (Wed, 11 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