[PATCH] D153550: [DWARFVerifier] Fix verification of nameless variables.
David Blaikie via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 26 11:07:52 PDT 2023
dblaikie added a comment.
In D153550#4448765 <https://reviews.llvm.org/D153550#4448765>, @fdeazeve wrote:
> In D153550#4441881 <https://reviews.llvm.org/D153550#4441881>, @dblaikie wrote:
>
>> But maybe the deeper question: Why are we creating DW_AT_variables for string literals? They aren't variables as far as a user would be concerned. GCC doesn't do this...
>
> @dblaikie apparently this was done last year: https://reviews.llvm.org/D123534
ah, right... - thanks for the reference! & then, yeah, just not putting these things in the index seems good. (even if we did revisit the original decision to include this debug info unconditionally - adding it under a flag, etc, we'd still want to fix the index issue)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153550/new/
https://reviews.llvm.org/D153550
More information about the llvm-commits
mailing list