[cfe-commits] r160231 - /cfe/trunk/lib/Driver/Tools.cpp

[Dimitry Andric]

On 2012-07-16 00:33, Anton Korobeynikov wrote:
>> I've just done a clean build on FreeBSD 8.3/x86-64 and this test passes.  I've also tested on 9.0-RELEASE and 10-CURRENT and I can't reproduce this test failure.  Is there something odd about this buildbot?
> http://lab.llvm.org:8011/buildslaves/kistanova7 says it's 8.2

Aha, FreeBSD 8.x still uses binutils 2.15, which does support the
--enable-new-dtags flag, but not --hash-style.  The test failure case
seems to try and build a hello.c.tmp.exe, and run it, but I guess the
actual link fails?  It isn't entirely clear from the log file.

So I added a check for major release, before using the --hash-style
option, to the patch I already had for cleaning up the FreeBSD linker
flags.  Please review the attached freebsd-link-flags-3.diff.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freebsd-link-flags-3.diff
Type: text/x-diff
Size: 2003 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120729/f987b7a5/attachment.diff>