[PATCH] D91670: Fix crash after looking up dwo_id=0 in CU index.

[Jorge Gorbe Moya via Phabricator via llvm-commits]

jgorbe added a comment.

Thanks everyone for the reviews! I'm committing this now.



================
Comment at: llvm/test/tools/llvm-symbolizer/split-dwarf-zero-signature-not-found.s:48
+        .section        .debug_addr,"", at progbits
+        .quad   A\I
+        .quad   F\I
----------------
jhenderson wrote:
> jgorbe wrote:
> > MaskRay wrote:
> > > `A\I` is undefined. Consider defining the symbol to be more realistic.
> > Good catch! Went a bit too far trimming the lldb test this is based on :)
> It's probably not applicable here, but FYI, yaml2obj now has some DWARF support which allows you to craft sections using YAML rather than assembly, without needing to specify all details. Take a look at some of the tests that are using it for examples of how you might use it.
Thanks for the tip! I'll have a look so I can use it in future tests :)


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

https://reviews.llvm.org/D91670