[PATCH] D76224: [llvm-objdump] Add call_frame_cfa locations to variable display

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 17 03:00:45 PDT 2020


jhenderson added inline comments.


================
Comment at: llvm/unittests/DebugInfo/DWARF/DWARFExpressionCompactPrinterTest.cpp:157
+TEST_F(DWARFExpressionCompactPrinterTest, Test_OP_fbreg_cfa) {
+  TestExprPrinter({DW_OP_fbreg, 0x78}, "[CFA-8]", {DW_OP_call_frame_cfa});
+}
----------------
I'm not up to speed on the DW_OP values. What does this value represent?

Assuming that the "0x78" is something to do with the -8, you probably want more values, and to test the bounds of whatever it can be.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76224





More information about the llvm-commits mailing list