[PATCH] D143463: [X86] Use the CFA when appropriate for better variable locations around calls.
Scott Linder via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 23 13:26:45 PDT 2023
scott.linder added inline comments.
================
Comment at: llvm/test/DebugInfo/X86/stack_adjustments_trigger_cfa_frame_base.ll:143-146
+attributes #0 = { inlinehint nonlazybind uwtable "probe-stack"="__rust_probestack" "target-cpu"="x86-64" }
+attributes #1 = { noinline nonlazybind uwtable "probe-stack"="__rust_probestack" "target-cpu"="x86-64" }
+attributes #2 = { nonlazybind uwtable "probe-stack"="__rust_probestack" "target-cpu"="x86-64" }
+attributes #3 = { nocallback nofree nosync nounwind readnone speculatable willreturn }
----------------
jmorse wrote:
> NB -- if these attributes aren't necessary for the test to operate, it's better to remove them to avoid future maintenance burdens
I removed these before landing!
@khuey let me know if you have concerns with dropping the attributes
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143463/new/
https://reviews.llvm.org/D143463
More information about the llvm-commits
mailing list