[all-commits] [llvm/llvm-project] a2b5fd: [SandboxIR][Region] Auxiliary vector metadata now ...

[vporpo via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a2b5fd7f6e06e1535582fe0358e1c1895a9fa9e5
      https://github.com/llvm/llvm-project/commit/a2b5fd7f6e06e1535582fe0358e1c1895a9fa9e5
  Author: vporpo <vporpodas at google.com>
  Date:   2025-05-27 (Tue, 27 May 2025)

  Changed paths:
    M llvm/include/llvm/SandboxIR/Region.h
    M llvm/lib/SandboxIR/Region.cpp
    M llvm/unittests/SandboxIR/RegionTest.cpp

  Log Message:
  -----------
  [SandboxIR][Region] Auxiliary vector metadata now requires a region (#137443)

The auxiliary vector is represented by the !sandboxaux metadata. But
until now adding an instruction to the aux vector would not
automatically add it to the region (i.e., it would not mark it with
!sandboxvec). This behavior was problematic when generating regions from
metadata, because you wouldn't know which region owns the auxiliary
vector.

This patch fixes this issue: We now require that an instruction with
!sandboxaux also defines its region with !sandboxvec.



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