[PATCH] D90507: Adding DWARF64 clang flag

Alexander Yermolovich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 18 12:38:58 PST 2020


ayermolo marked 2 inline comments as done.
ayermolo added inline comments.


================
Comment at: clang/include/clang/Basic/CodeGenOptions.def:35
 CODEGENOPT(AsmVerbose        , 1, 0) ///< -dA, -fverbose-asm.
+CODEGENOPT(Dwarf64           , 1, 0) ///< -gdwarf64.
 CODEGENOPT(PreserveAsmComments, 1, 1) ///< -dA, -fno-preserve-as-comments.
----------------
ikudrin wrote:
> dblaikie wrote:
> > Is there any precedent to draw from for this flag name? (Does GCC support DWARF64? Does it support it under this flag name or some other? (similarly with other gcc-like compilers (Intel's? Whoever else... )))
> It looks like we are pioneering in that area. To me, the proposed name looks consonant with other debug-related switches.
I didn't see any dwarf64 flags in gcc:
https://gcc.gnu.org/onlinedocs/gcc/Option-Summary.html

I tried to follow clang convention for other dwarf flags.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90507



More information about the llvm-commits mailing list