[llvm] [SandboxIR][Region] Auxiliary vector metadata now requires a region (PR #137443)

via llvm-commits llvm-commits at lists.llvm.org
Fri May 23 17:02:21 PDT 2025


vporpo wrote:

> The TrackScore changes look unrelated to the rest of the patch.

Yeah it looks unrelated, but it is a workaround to avoid affecting the region's cost when adding an instruction to an aux vector. Let me explain: The way we calculate the region's total instruction cost at the moment is by keeping track of all the instructions added to the region. But with this patch we want any instruction in an aux vector to also be in a region. But doing that would affect the region's cost, which is undesirable.

We will eventually move away from this region-based cost tracking, which should clean this up.


https://github.com/llvm/llvm-project/pull/137443


More information about the llvm-commits mailing list