[all-commits] [llvm/llvm-project] d74290: Delegate to ABI plugin to check if call frame addr...

[pveras via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d74290319e3db3425bf2f0f87ef6c32f1078371f
      https://github.com/llvm/llvm-project/commit/d74290319e3db3425bf2f0f87ef6c32f1078371f
  Author: pveras <verasbs at gmail.com>
  Date:   2025-10-13 (Mon, 13 Oct 2025)

  Changed paths:
    M lldb/source/Target/RegisterContextUnwind.cpp

  Log Message:
  -----------
  Delegate to ABI plugin to check if call frame addresses are valid (#161398)

Specially when dealing with different address spaces, CFAs could start
from addresses like 0. For instance, Nvidia GPUs have instructions to
read from local memory that use 0-based offsets and stack memory can be
referenced by these offsets rather than global addresses. Note that ABIs
could already specify what they consider to be valid CFA values but this
was never used in these parts of the unwinder code. For most ABIs, this
makes the validation more strict, as they already used to discard 0s and
then checked for alignment which would discard 1s. There a few
exceptions where 0s were possible and this makes it less strict, like
the RISCV and ARC ABIs.

@jasonmolenda Would you be the appropriate reviewer for this?
Also cc. @clayborg @walter-erquinigo



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