On Wed, May 4, 2022 at 11:26 AM Pavel Labath via Phabricator <
revi...@reviews.llvm.org> wrote:
> labath accepted this revision.
> labath added a comment.
> This revision is now accepted and ready to land.
>
> Looks good. Thanks for the patch. Do you need me to commit this for you?
>
Yes please - thanks!
>
> Left two comments inline, but I can also fix that when I'm pushing it.
>
That'd be awesome. I played with dropping the main() function, but that
seems to weird lldb out somehow, and I don't get it reading the value of
ug.m.
>
>
>
> ================
> Comment at:
> lldb/test/Shell/SymbolFile/DWARF/x86/debug_static-member-anonymous-namespace.s:5
> +# CHECK: (lldb) target variable ug
> +# CHECK: (U) ug = (m = 0)
> +
> ----------------
> Better use some other number here, as it's fairly easy for a value to
> accidentally come out as zero.
>
>
> ================
> Comment at:
> lldb/test/Shell/SymbolFile/DWARF/x86/debug_static-member-anonymous-namespace.s:22-25
> +#
> +# int main() {
> +# return ug.m;
> +# }
> ----------------
> We're not going to be running this, so you don't really need the main
> function.
>
>
> Repository:
> rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
> https://reviews.llvm.org/D124409/new/
>
> https://reviews.llvm.org/D124409
>
>
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
