<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 23, 2014 at 4:30 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank" class="cremed">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Looks good - though I'd prefer to separate the discussion about column info so as to isolate the changes/tests/etc and not delay this review.<br>

<div class=""><br>
================<br>
Comment at: lib/Frontend/CompilerInvocation.cpp:571<br>
@@ +570,3 @@<br>
+      Opts.setDebugInfo(CodeGenOptions::LocTrackingOnly);<br>
+      Opts.DebugColumnInfo = true;<br>
+    }<br>
----------------<br>
</div>Perhaps we could leave this out for now & just have a separate CR/discussion about the semantics/ramifications of having this difference between -R and -gmlt?<br></blockquote><div><br></div><div>Hm, perhaps. I would be penalizing -Rpass, however. But I guess it doesn't matter much. I'll amend the documentation to state that -gcolumn-inf is needed with -Rpass.</div>
<div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
================<br>
Comment at: test/Frontend/optimization-remark.c:13<br>
@@ +12,3 @@<br>
<div class="">+// CHECK: , !dbg !<br>
</div>+// CHECK-NOT: DW_TAG_base_type<br>
+<br>
----------------<br>
Could have a brief comment here "Ensure -Rpass, like -gmlt, doesn't produce debug info metadata for types".<br></blockquote><div><br></div><div>That's what the comment above those two lines tries to do. Perhaps you want it worded differently?</div>
</div></div></div>