[PATCH] D46021: [DEBUGINFO] Disable emission of the dwarf sections, but allow directives.

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 24 10:05:01 PDT 2018


dblaikie added a comment.

Sorry, I'm a bit confused.

1. I'd expect /this/ behavior (line table directives, but no debug_info, etc) to be controlled via a DICompileUnit emission kind (a value between "NoDebug" and "LineTablesOnly" as Paul mentioned).

2. I'd expect the "no ', debug'" flag (which I don't see in this change? Is it missing? is it elsewhere?) to be controlled via an MCOption - because it must be global.

Now, arguably, if we have the global "no ', debug'" flag we maybe might as well use it to implement this behavior too since we don't have any other use for it? So maybe (2) invalidates (1).


Repository:
  rL LLVM

https://reviews.llvm.org/D46021





More information about the llvm-commits mailing list