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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
