[PATCH] D81131: [DebugInfo] Fix assertion for extern void type

Yonghong Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 4 22:33:51 PDT 2020


yonghong-song added a comment.

Thanks, @dblaikie

>> I think if we're going to support "const void" we should support "void" too & fix the verifier to allow this & make sure LLVM produces the correct debug info for it (which is probably a DW_TAG_variable without a DW_AT_type attribute (same as a function with a void return type has no DW_AT_type attribute) - and check that GCC does the same thing.

Your suggestion makes sense, I will try to fix the fatal error according to what you suggested.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81131





More information about the cfe-commits mailing list