<div dir="ltr">You could handle some of the DW_AT_APPLE foo things that way?<br><br>-eric<br></div><br><div class="gmail_quote">On Wed, Mar 25, 2015 at 2:47 PM Paul Robinson <<a href="mailto:Paul_Robinson@playstation.sony.com">Paul_Robinson@playstation.sony.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">================<br>
Comment at: include/clang/Driver/Options.<u></u>td:1051<br>
@@ -1047,1 +1050,3 @@<br>
+def gsce : Flag<["-"], "gsce">, Group<g_flags_Group>,<br>
+  HelpText<"Tune debug info for an SCE target (e.g. PS4)">, Flags<[CC1Option]>;<br>
 def gdwarf_2 : Flag<["-"], "gdwarf-2">, Group<g_Group>,<br>
----------------<br>
dblaikie wrote:<br>
> Is SCE usually pronounced with each letter separately, or as a word ("ski" or similar)? (wondering whether "an SCE target" or "a SCE target" is correct)<br>
SCE = ess see ee (Sony Computer Entertainment)<br>
There is a European division, SCEE, which I have heard pronounced as "ski."<br>
<br>
<br>
================<br>
Comment at: include/clang/Frontend/<u></u>CodeGenOptions.def:159<br>
@@ +158,3 @@<br>
+/// Which debugger we are targeting.<br>
+ENUM_CODEGENOPT(Debugger, llvm::DebuggerKind, 2, llvm::DebuggerKind::GDB)<br>
+<br>
----------------<br>
dblaikie wrote:<br>
> I take it that GDB is the default here? But ignored in the frontend or driver in favor of platform-specific defaults?<br>
Well, it seems to be customary to put a default here, and then initialize the field explicitly elsewhere anyway without relying on the default value.  I didn't exhaustively check every field, but eyeballing a handful it was certainly the pattern.<br>
The most entertaining case was DwarfVersion.<br>
<br>
================<br>
Comment at: lib/Frontend/<u></u>CompilerInvocation.cpp:408<br>
@@ -407,1 +407,3 @@<br>
<br>
+  if (Args.hasArg(OPT_ggdb))<br>
+    Opts.setDebugger(llvm::<u></u>DebuggerKind::GDB);<br>
----------------<br>
dblaikie wrote:<br>
> Should this handling be up in the driver? Then it should be easy to write tests for the defaults.<br>
No, it should be here, so we can specify a debugger target for -cc1 tests.<br>
There aren't any use-cases for that yet, but Katya's import-anonymous-namespace patch could use it, once it goes in.<br>
<br>
<a href="http://reviews.llvm.org/D8599" target="_blank">http://reviews.llvm.org/D8599</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
<br>
<br>
</blockquote></div>