[all-commits] [llvm/llvm-project] ce786f: 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: ce786fdf6db8ec2202dea3398bf888ac8b60cc76
      https://github.com/llvm/llvm-project/commit/ce786fdf6db8ec2202dea3398bf888ac8b60cc76
  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