<div dir="ltr">(And sorry for the breakage!  I didn't see any nag mails).</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 6, 2016 at 7:34 AM, Todd Fiala <span dir="ltr"><<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">CFLAGS_EXTRA didn't work because it then gets both -O0 and -O1 (for the majority of our tests, Makefile.rules correctly assumes optimizations are turned off for maximal debug info).  If that was the fix, that is circumventing the check this test is doing.<div><br></div><div>What we really need (and I'll go back and add) is an explicit optimization level variable for Makefile.rules that defaults to -O0 but allows overriding.  That would most likely address this.  I'll go back to that once I'm done with another task.</div><div><br></div><div>Thanks!</div><div><br></div><div>-Todd</div></div><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">On Wed, Apr 6, 2016 at 7:27 AM, Tamas Berghammer <span dir="ltr"><<a href="mailto:tberghammer@google.com" target="_blank">tberghammer@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tberghammer added a comment.<br>
<br>
My guess is the root cause of the problem is that we are setting CFLAGS instead of appending to CLAFGS_EXTRA (I changed it in my followup CL)<br>
<div><div><br>
<br>
Users:<br>
  tfiala (Author)<br>
<br>
<a href="http://reviews.llvm.org/rL265498" rel="noreferrer" target="_blank">http://reviews.llvm.org/rL265498</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div><div dir="ltr">-Todd</div></div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">-Todd</div></div>
</div>