This revision was automatically updated to reflect the committed changes.
Closed by commit rL341212: [DEBUGINFO] Add support for emission of the debug
directives only. (authored by ABataev, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.
Not sure that every test for line-tables-only needs to also test
line-directives-only, but not a huge deal either way. (only the cases where
there's actually a different codepath to
ABataev updated this revision to Diff 163353.
ABataev added a comment.
Address comment from David.
Repository:
rC Clang
https://reviews.llvm.org/D51177
Files:
include/clang/Basic/DebugInfoOptions.h
include/clang/Driver/Options.td
lib/CodeGen/CGDebugInfo.cpp
dblaikie added inline comments.
Comment at: test/Driver/debug-options.c:259
//
+// This tests asserts that "-glineinfo-only" "-g0" disables debug info.
+// GLIO_NO: "-cc1"
'lineinfo' seems like two words - the inconsistency with 'line-tables' seems
like it'd
ABataev added a comment.
In https://reviews.llvm.org/D51177#1213079, @echristo wrote:
> Should we just have them mean the same thing and change it based on target?
I reused the same debug info level support in clang, but in LLVM they have
different processing and I'm not sure that they are
echristo added a comment.
Should we just have them mean the same thing and change it based on target?
Repository:
rC Clang
https://reviews.llvm.org/D51177
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
ABataev created this revision.
ABataev added a reviewer: echristo.
Herald added subscribers: JDevlieghere, fedor.sergeev, aprantl.
Added option -glineinfo-only to support emission of the debug directives
only. It behaves very similar to -gline-tables-only, except that it sets
llvm debug info