[PATCH] D59518: [DwarfDebug] Skip entries to big for 16 bit size field in Dwarf < 5.

[Paul Robinson via Phabricator via llvm-commits]

probinson added a comment.

Generating the test on the fly with Python seems plausible.  Note you don't need 16K components, just enough components to make the location description exceed 16K bytes. If each component is (say) a 64-bit field with a max-int constant value, that's something like 11 bytes of description per field, so ~1500 fields ought to do it.



================
Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp:1987
+    // The entry is too big to fit into 16 bit, drop it as there is nothing we
+    // can do.
+    return;
----------------
We have to emit a zero length here, otherwise the location-list entry will be syntactically incorrect.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59518