> 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
This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:59 UTC