I'm glad you asked - I got a positive result the first time... But your question coincided with my machine grinding to a halt. After a reboot and rebuild of code base - no it doesn't pass. I needed to include the flags added by the driver viz: "-fno-signed-char -fno-common" (these are tested for in test/Driver/xcore-opts.c)
However, "-triple xcore" is fine... but I have changed it to "-triple xcore-unknown-unknown" for consistency. -// RUN: %clang -target xcore -O0 -o - -emit-llvm -S %s | FileCheck %s +// RUN: %clang_cc1 -triple xcore-unknown-unknown -fno-signed-char -fno-common -emit-llvm -o - %s | FileCheck %s Thank you Robert ________________________________________ From: Dmitri Gribenko [[email protected]] Sent: 22 August 2013 09:09 To: Robert Lytton Cc: cfe-commits cfe Subject: Re: r188663 - XCore target: correct test layout On Thu, Aug 22, 2013 at 1:01 AM, Robert Lytton <[email protected]> wrote: > Hi Dmitri, > > Here is a patch to correct test RUN parameters. Does it pass tests? (I did not try.) The -triple does not look like a triple (for example, x86_64-unknown-unknown). Should it be xcore-unknown-unknown? Dmitri -- main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if (j){printf("%d\n",i);}}} /*Dmitri Gribenko <[email protected]>*/
PatchTestRunParams
Description: PatchTestRunParams
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
