[all-commits] [llvm/llvm-project] 9a8da8: RegAllocGreedy: Add dummy priority advisor for wri...

[Matt Arsenault via All-commits]

  Branch: refs/heads/users/arsenm/greedy-add-dummy-priority-advisor
  Home:   https://github.com/llvm/llvm-project
  Commit: 9a8da8ee4572a817b9743e7ff6bc2485b614470a
      https://github.com/llvm/llvm-project/commit/9a8da8ee4572a817b9743e7ff6bc2485b614470a
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2024-12-27 (Fri, 27 Dec 2024)

  Changed paths:
    M llvm/lib/CodeGen/RegAllocGreedy.cpp
    M llvm/lib/CodeGen/RegAllocPriorityAdvisor.cpp
    M llvm/lib/CodeGen/RegAllocPriorityAdvisor.h
    A llvm/test/CodeGen/AMDGPU/dummy-regalloc-priority-advisor.mir

  Log Message:
  -----------
  RegAllocGreedy: Add dummy priority advisor for writing MIR tests

I regularly struggle reproducing failures in greedy due to changes
in priority when resuming the allocation from MIR vs. a complete
compilation starting at IR. That is, the fix in
e0919b189bf2df4f97f22ba40260ab5153988b14 did not really fix the
problem of the instruction distance mattering.

Add a way to bypass all of the priority heuristics for MIR tests,
by prioritizing only by virtual register number. Could also
give this a more specific name, like PrioritizeLowVirtRegNumber



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