[cfe-commits] Update Cortex-A9 buildslaves

[Galina Kistanova]

Hi David,

Thank you for the analyses.
I'll change the builder configuration today to remove --with-abi=aapcs
param and use armv7l-unknown-linux-gnueabihf.


Thanks

Galina



On Thu, Sep 27, 2012 at 9:45 AM, David Tweed <david.tweed at arm.com> wrote:
> OK, some investigation results from the always invigorating world of ABI
> configuration issues:
>
> If you are producing an armv7l-unknown-linux-gnueabihf target then you NEED
> TO DEFINITELY NOT USE --abi=aapcs to avoid some FileCheck problems.
> However, if you are producing an armv7l-unknown-linux-gnueabi target then
> you NEED --abi=aapcs to avoid some problems.
>
> On a recent/future ubuntu ARM based system, it's correct to specify
> armv7l-unknown-linux-gnueabihf and so we should use that and drop the
> --abi=aapcs entry. (It's still an open question whether in other
> circumstances plain gnueabi remains an interesting target that ought to be
> tested, but on stock ubuntu gnueabihf is the way to go.) If I do that on my
> pandaboard I both get rid of some of the existing failures and I don't
> create any new ones. I've also tested clang and it produces runnable
> executables. So I think this change is what ought to be there.
>
> Could I ask you to try that configuration options on your buildslaves and
> see if that reproduces what I see? Once it's clear this is ok I'll try to do
> a formal patch to the ARM installation notes in LLVM.
>
> (I don't think aapcs causes "real-life" problems, it just changes function
> signatures which FileCheck regexps doesn't understand.)
>
> Many thanks, and sorry for the confusion.
>
> Cheers,
> David
>
> -----Original Message-----
> From: David Tweed [mailto:david.tweed at arm.com]
> Sent: 27 September 2012 11:21
> To: David Tweed; 'Galina Kistanova'; 'llvm-commits at cs.uiuc.edu'; 'llvm cfe'
> Cc: Amara Emerson; Kristof Beyls
> Subject: RE: [cfe-commits] Update Cortex-A9 buildslaves
>
> Right, the situation is more complex than I thought. Using the same triple
> as the ubuntu gcc was compiled with fixes (in the "right" way) some of the
> regression test failures. However, a just completed run of the full
> regression tests has shown it causes about 50 new failures within clang. I
> need to look at these to see why this change has occurred, but I think it's
> probably best not to change the triple just yet.
>
> In general, while the interaction between the clang driver search mechanism
> and the pre-existing gcc installation is, I suspect, always going to be
> complex and ad-hoc, at the moment there's no integrated gives no feedback
> about how all these checks have influenced the paths it knows about, etc.
> I'm not sure if this is something worth trying to address.
>
> Thanks,
> David
>
> -----Original Message-----
> From: David Tweed [mailto:david.tweed at arm.com]
> Sent: 27 September 2012 10:03
> To: 'Galina Kistanova'; llvm-commits at cs.uiuc.edu; llvm cfe
> Cc: Amara Emerson; Kristof Beyls
> Subject: RE: [cfe-commits] Update Cortex-A9 buildslaves
>
> Hi, I haven't quite completed my internal testing yet, but since you're
> making modifications to the buildbots:
>
> according to
>
> http://lab.llvm.org:8011/builders/clang-native-arm-cortex-a9/builds/3118/ste
> ps/configure/logs/stdio
>
> you're configuring with armv7l-unknown-linux-gnueabi. Could you change that
> to armv7l-unknown-linux-gnueabihf please? I've been digging into some clang
> failures and I think that this configuration used to match the debian/ubuntu
> gcc configuration, then at some point debian/ubuntu changed their
> configuration, especially for gcc. clang particularly depends on extracting
> info from the gcc configuration, and with a mismatched triple it thinks
> there's not a working gcc installation so stops lots of stuff early.
>
> Tests are still running, but I think it solves some clang issues and doesn't
> introduce any new ones.
>
> Let me know if any of this is unclear
>
> Cheers
>
> -----Original Message-----
> From: cfe-commits-bounces at cs.uiuc.edu
> [mailto:cfe-commits-bounces at cs.uiuc.edu] On Behalf Of Galina Kistanova
> Sent: 26 September 2012 19:36
> To: llvm-commits at cs.uiuc.edu; llvm cfe
> Subject: [cfe-commits] Update Cortex-A9 buildslaves
>
> Hello everyone,
>
> We have updated both Cortex-A9 buildslaves to reduce the number of
> failing tests.
> Currently both Pandaboards run Ubuntu 12.10 and the following
> development toolchain:
>
> gcc (Ubuntu/Linaro 4.7.2-1ubuntu1) 4.7.2
> g++ (Ubuntu/Linaro 4.7.2-1ubuntu1) 4.7.2
> GNU ld (GNU Binutils for Ubuntu) 2.22.90.20120919
> Python 2.7.3
>
> Please note that the current expected number of failing tests on this
> builder is 17.
> We are working on fixing them, and as always, patches are welcome.
>
>
> Thanks
>
> Galina
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
>
>