[PATCH] D112286: Add -dwarf-inlined-strings=Disable option to DWARF tests that expect DW_FORM_strp
ChenZheng via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 25 18:46:26 PDT 2021
shchenz added a comment.
In D112286#3086128 <https://reviews.llvm.org/D112286#3086128>, @Jake-Egan wrote:
> In D112286#3085048 <https://reviews.llvm.org/D112286#3085048>, @dblaikie wrote:
>
>> 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)
>
> I agree generalizing the test is probably the better option if `DW_FORM_strp` isn't necessary for the purpose of the test.
>
> @shchenz I will update the patch if you have no objection.
Sounds good to me
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