[PATCH] D95518: [Debug-Info][XCOFF] support dwarf for XCOFF for assembly output

[ChenZheng via Phabricator via llvm-commits]

shchenz added a comment.

Thanks for review @jmorse  and thanks for explanation @hubert.reinterpretcast

Yes, we still have some work to do for object output for XCOFF 64-bit. So before that is done, we have to test the assembly output for debug infos. 
Also yes, I plan to add `llvm-dwarfdump` test for the newly added `empty.ll` when I post the object output support for DWARF of XCOFF.



================
Comment at: llvm/test/DebugInfo/XCOFF/empty.ll:18
+
+attributes #0 = { noinline nounwind optnone "disable-tail-calls"="false" "frame-pointer"="all" "less-precise-fpmad"="false" "min-legal-vector-width"="0" "no-infs-fp-math"="false" "no-jump-tables"="false" "no-nans-fp-math"="false" "no-signed-zeros-fp-math"="false" "no-trapping-math"="true" "stack-protector-buffer-size"="8" "target-cpu"="pwr4" "target-features"="-altivec,-bpermd,-crypto,-direct-move,-extdiv,-float128,-htm,-power8-vector,-power9-vector,-spe,-vsx" "unsafe-fp-math"="false" "use-soft-float"="false" }
+
----------------
jmorse wrote:
> NB: we usually delete any un-necessary attributes (normally all of them) for debug-info tests, as they're otherwise a needless maintenance burden.
Good idea. Thanks for reminder.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95518