Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <cb071db4e9deecbcadf9b32cf7b64e2f@smtp.hushmail.com>
Date: Sun, 20 Jan 2013 11:00:40 +0100
From: magnum <john.magnum@...hmail.com>
To: john-dev@...ts.openwall.com
Subject: Re: new clang 3.2 warnings

On 20 Jan, 2013, at 10:55 , magnum <john.magnum@...hmail.com> wrote:
> On 20 Jan, 2013, at 10:38 , Frank Dittrich <frank_dittrich@...mail.com> wrote:
>> clang: warning: argument unused during compilation: '-faddress-sanitizer'
>> 
>> I just assumed that my clang version was too old to know this switch.
>> Since only the -clang-debug target was affected, I didn't care.
>> 
>> I would prefer -clang-debug just using the option name that is required
>> for the newest clang version(s), instead of having to adjust one more
>> config variable.
>> But when I change the option name in Makefile, I get
>> clang: warning: argument unused during compilation: '-fsanitize=address'
>> instead, for 2.9 and for 3.1.
>> 
>> This is bad. May be HAVE_CLANG_3_2 is a good enough solution right now?
>> How widespread is clang 3.2? Fedora 18 still uses 3.1.
> 
> Bull's clang is 3.0-6ubuntu3. It doesn't seem to understand either of the variants.

No, wait. The message "argument unused during compilation" just tells us that option should be in LDFLAGS only, not CFLAGS. So Bull's clang does support -fsanitize-address. I'll commit a patch.

magnum

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.