(And sorry for the breakage!  I didn't see any nag mails).

On Wed, Apr 6, 2016 at 7:34 AM, Todd Fiala <todd.fi...@gmail.com> wrote:

> 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.
>
> 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.
>
> Thanks!
>
> -Todd
>
> On Wed, Apr 6, 2016 at 7:27 AM, Tamas Berghammer <tbergham...@google.com>
> wrote:
>
>> tberghammer added a comment.
>>
>> 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)
>>
>>
>> Users:
>>   tfiala (Author)
>>
>> http://reviews.llvm.org/rL265498
>>
>>
>>
>>
>
>
> --
> -Todd
>



-- 
-Todd
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to