[PATCH] D100628: [Debug-Info][DBX] DW_AT_noreturn should not be generated when dwarf version is not 5

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 23 09:58:16 PDT 2021


aprantl added inline comments.


================
Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp:1281
+  if (SP->isNoReturn() &&
+      (!Asm->TM.Options.DebugStrictDwarf || DD->getDwarfVersion() >= 5))
     addFlag(SPDie, dwarf::DW_AT_noreturn);
----------------
Should we factor out a FlagSupported(attribute) function? I think Dwarf.def already stores the version in which an attribute was introduced, so we don't need to duplicate this in the code here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100628



More information about the llvm-commits mailing list