LGTM

On 16 August 2013 04:36, Robert Lytton <[email protected]> wrote:
> Hi Rafael,
>
> Here is the amended patch.
> I've moved IL checks into test/CodeGen/xcore-abi.c.
> Also built at -O0, amending CHECK statements to be less brittle.
>
> Robert
>
> ________________________________________
> From: Rafael Espíndola [[email protected]]
> Sent: 16 August 2013 00:53
> To: Robert Lytton
> Cc: [email protected]
> Subject: Re: XCore make test checks more concise
>
> On 15 August 2013 11:29, Robert Lytton <[email protected]> wrote:
>> (http://clang.llvm.org/docs/DriverInternals.html)
>> Now I get it!
>>
>> So test should be:
>> // RUN: %clang -target xcore %s -### -o %t.o 2>&1 | FileCheck  %s
>> // CHECK: "-momit-leaf-frame-pointer"
>> // CHECK: "-fno-signed-char"
>> // CHECK: "-fno-common"
>
> Yes.
>
>> I'm a bit bothered that there is no "-fomit-frame-pointer".
>> Should I see this too?
>
> It is mdisable-fp-elim in the cc1 level.
>
> Cheers,
> Rafael

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to