[PATCH] D143463: [X86] Use the CFA when appropriate for better variable locations around calls.

[Kyle Huey via Phabricator via llvm-commits]

khuey 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 }
----------------
scott.linder wrote:
> 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
Should be fine, I believe.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143463/new/

https://reviews.llvm.org/D143463