[llvm-commits] [llvm] r168316 - in /llvm/trunk/test/ExecutionEngine: MCJIT/lit.local.cfg lit.local.cfg

[Hal Finkel]

----- Original Message -----
> From: "NAKAMURA Takumi" <geek4civic at gmail.com>
> To: "Ulrich Weigand" <ulrich.weigand at de.ibm.com>
> Cc: "llvm-commits" <llvm-commits at cs.uiuc.edu>, "Hal Finkel" <hfinkel at anl.gov>
> Sent: Monday, November 19, 2012 10:45:55 PM
> Subject: Re: [llvm-commits] [llvm] r168316 - in /llvm/trunk/test/ExecutionEngine: MCJIT/lit.local.cfg lit.local.cfg
> 
> It broke ppc32-linux (f12 ps3).
> 
> http://bb.pgr.jp/builders/clang-ppc-linux/builds/693

Thanks! We suspected this might happen. We thought, however, that it was better to show the new MCJIT infrastructure working on PPC64 than to show the deprecated JIT infrastructure functional on PPC32. Having testing coverage on MCJIT is more important now.

Do you know if there is a way to make this change specific to PPC64?

 -Hal

> 
> 2012/11/20 Ulrich Weigand <ulrich.weigand at de.ibm.com>:
> > Author: uweigand
> > Date: Mon Nov 19 11:57:07 2012
> > New Revision: 168316
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=168316&view=rev
> > Log:
> > Enable MCJIT tests on PowerPC.
> > Disable old JIT tests on PowerPC.
> >
> > Modified:
> >     llvm/trunk/test/ExecutionEngine/MCJIT/lit.local.cfg
> >     llvm/trunk/test/ExecutionEngine/lit.local.cfg
> >
> > Modified: llvm/trunk/test/ExecutionEngine/MCJIT/lit.local.cfg
> > URL:
> > http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/MCJIT/lit.local.cfg?rev=168316&r1=168315&r2=168316&view=diff
> > ==============================================================================
> > --- llvm/trunk/test/ExecutionEngine/MCJIT/lit.local.cfg (original)
> > +++ llvm/trunk/test/ExecutionEngine/MCJIT/lit.local.cfg Mon Nov 19
> > 11:57:07 2012
> > @@ -8,12 +8,13 @@
> >  root = getRoot(config)
> >
> >  targets = set(root.targets_to_build.split())
> > -if ('X86' in targets) | ('ARM' in targets) | ('Mips' in targets):
> > +if ('X86' in targets) | ('ARM' in targets) | ('Mips' in targets) |
> > \
> > +   ('PowerPC' in targets):
> >      config.unsupported = False
> >  else:
> >      config.unsupported = True
> >
> > -if root.host_arch not in ['x86', 'x86_64', 'ARM', 'Mips']:
> > +if root.host_arch not in ['x86', 'x86_64', 'ARM', 'Mips',
> > 'PowerPC']:
> >      config.unsupported = True
> >
> >  if root.host_os in ['Darwin']:
> >
> > Modified: llvm/trunk/test/ExecutionEngine/lit.local.cfg
> > URL:
> > http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/lit.local.cfg?rev=168316&r1=168315&r2=168316&view=diff
> > ==============================================================================
> > --- llvm/trunk/test/ExecutionEngine/lit.local.cfg (original)
> > +++ llvm/trunk/test/ExecutionEngine/lit.local.cfg Mon Nov 19
> > 11:57:07 2012
> > @@ -1 +1,12 @@
> >  config.suffixes = ['.ll', '.c', '.cpp']
> > +
> > +def getRoot(config):
> > +    if not config.parent:
> > +        return config
> > +    return getRoot(config.parent)
> > +
> > +root = getRoot(config)
> > +
> > +if root.host_arch in ['PowerPC']:
> > +    config.unsupported = True
> > +
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 

-- 
Hal Finkel
Postdoctoral Appointee
Leadership Computing Facility
Argonne National Laboratory