[PATCH] D112286: Add -dwarf-inlined-strings=Disable option to DWARF tests that expect DW_FORM_strp

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 25 10:42:18 PDT 2021


dblaikie added a comment.

I think probably generalizing the tests to not care about the string encodings is probably the right thing, rather than forcing indirect strings?

I think these tests are overly specific - they don't look like they should care about the string encoding (& Probably an artifact of older techniques in writing these debug info tests - improvements to the dumper to print strings inline, skip forms, etc)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112286



More information about the llvm-commits mailing list