<br><br><div class="gmail_quote">On Sun, Apr 29, 2012 at 3:07 AM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@apple.com" target="_blank">echristo@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div class="im"><div><blockquote type="cite"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>Sending the revised (modulo r155697) patch. Please take a look.</div><div>
<br></div><div>Codereview: <a href="http://codereview.appspot.com/6112056/#ps9001" target="_blank">http://codereview.appspot.com/6112056/#ps9001</a></div>
</div></div></blockquote><br></div></div><div>I think about the only comment I've got is whether or not we should use comparison operators instead of == and != in these sorts of cases?</div><div><br></div><div><div>-    Dbg->EmitGlobalVariable(E->getDecl(), Init);</div>
<div>+    if (CGM.getCodeGenOpts().DebugInfo != CodeGenOptions::DebugLineTablesOnly)</div><div>+      Dbg->EmitGlobalVariable(E->getDecl(), Init);</div><div><br></div></div><div>I'd prefer it personally. Your thoughts?</div>
</div></blockquote><div><br></div><div>Yes, the comparison operators look better. Using them now.</div><div>See attachment and <a href="http://codereview.appspot.com/6112056/#ps11001">http://codereview.appspot.com/6112056/#ps11001</a> </div>
</div><div><br></div>-- <br><div>Alexey Samsonov, MSK</div><br>