r189884 - Use -O0 in this test too.

David Blaikie dblaikie at gmail.com
Tue Sep 3 18:52:52 PDT 2013


I'm not going to look through them all, but I assume it was a
find-and-replace of any -O0 on a clang_cc1 line? Seems reasonable to
me. (could be worth checking that no edited line also has a -ON line
on it, in case something tried to set it, then default it, etc.

On Tue, Sep 3, 2013 at 6:50 PM, Rafael EspĂ­ndola
<rafael.espindola at gmail.com> wrote:
> Sure. Is the attached patch OK?
>
> On 3 September 2013 19:36, David Blaikie <dblaikie at gmail.com> wrote:
>> Can you just drop the -O entirely?
>>
>> On Tue, Sep 3, 2013 at 4:31 PM, Rafael Espindola
>> <rafael.espindola at gmail.com> wrote:
>>> Author: rafael
>>> Date: Tue Sep  3 18:31:28 2013
>>> New Revision: 189884
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=189884&view=rev
>>> Log:
>>> Use -O0 in this test too.
>>>
>>> Modified:
>>>     cfe/trunk/test/CodeGenCXX/vtable-available-externally.cpp
>>>
>>> Modified: cfe/trunk/test/CodeGenCXX/vtable-available-externally.cpp
>>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGenCXX/vtable-available-externally.cpp?rev=189884&r1=189883&r2=189884&view=diff
>>> ==============================================================================
>>> --- cfe/trunk/test/CodeGenCXX/vtable-available-externally.cpp (original)
>>> +++ cfe/trunk/test/CodeGenCXX/vtable-available-externally.cpp Tue Sep  3 18:31:28 2013
>>> @@ -1,4 +1,4 @@
>>> -// RUN: %clang_cc1 %s -I%S -triple=x86_64-apple-darwin10 -emit-llvm -O3 -o %t
>>> +// RUN: %clang_cc1 %s -I%S -triple=x86_64-apple-darwin10 -emit-llvm -O0 -o %t
>>>  // RUN: FileCheck --check-prefix=CHECK-TEST1 %s < %t
>>>  // RUN: FileCheck --check-prefix=CHECK-TEST2 %s < %t
>>>  // RUN: FileCheck --check-prefix=CHECK-TEST5 %s < %t
>>>
>>>
>>> _______________________________________________
>>> cfe-commits mailing list
>>> cfe-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits




More information about the cfe-commits mailing list