11-CURRENT build fail with base gcc

[Warner Losh]

> On Aug 20, 2015, at 1:21 PM, Warner Losh <imp at bsdimp.com> wrote:
> 
> 
>> On Aug 20, 2015, at 10:32 AM, Dimitry Andric <dim at freebsd.org> wrote:
>> 
>> On 20 Aug 2015, at 18:24, Warner Losh <imp at bsdimp.com> wrote:
>>> 
>>>> On Aug 20, 2015, at 9:38 AM, Oliver Pinter <oliver.pinter at hardenedbsd.org> wrote:
>> ...
>>> 
>>> I think you are wrong about the cause. -mno-avx is bogusly listed unconditionally
>>> in efi/Makefile.inc. I’m working on a patch now…
>> 
>> Ah, this should be replaced with the recently introduced CFLAGS_NO_SIMD
>> variable, then?
> 
> Perhaps. Didn’t know this was a thing. That could use useful many places, though
> there were two clang specific args I needed to move, not just the one that’s in this
> flag. Maybe things are over-specified?
> 
> Not sure I like bsd.cpu.mk growing more name-space pollution, especially stuff that
> isn’t documented somewhere (bsd.cpu.mk is included from sys.mk, which is automaticallyed
> globally included). All these hacks being stashed hither and yon are starting to get
> very hard to keep straight for someone who looks at this code every day, let alone
> somebody who invested CFLAGS_NO_SIMD independently for their code and finds
> that breaks in an upgrade...

Ah, it doesn’t include -mno-aes, but luckily that’s understood by both. Not sure why that’s
needed, but is that the same ‘class’ or no?

Warner

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 842 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://lists.freebsd.org/pipermail/freebsd-current/attachments/20150820/a8d0f493/attachment.bin>