Re: [PATCH] i386: Enable AVX/AVX512 features only if supported by OSXSAVE

2018-03-30 Thread Ilya Verbin
2018-03-30 20:56 GMT+03:00 H.J. Lu : > On Fri, Mar 30, 2018 at 10:19 AM, Ilya Verbin wrote: >> This check will always disable AVX-512 on macOS, because they >> implemented on-demand support: >>

Re: [PATCH] i386: Enable AVX/AVX512 features only if supported by OSXSAVE

2018-03-30 Thread H.J. Lu
On Fri, Mar 30, 2018 at 10:19 AM, Ilya Verbin wrote: > This check will always disable AVX-512 on macOS, because they > implemented on-demand support: > https://github.com/apple/darwin-xnu/blob/0a798f6738bc1db01281fc08ae024145e84df927/osfmk/i386/fpu.c#L176 > Isn't xsaveopt

Re: [PATCH] i386: Enable AVX/AVX512 features only if supported by OSXSAVE

2018-03-30 Thread Ilya Verbin
This check will always disable AVX-512 on macOS, because they implemented on-demand support: https://github.com/apple/darwin-xnu/blob/0a798f6738bc1db01281fc08ae024145e84df927/osfmk/i386/fpu.c#L176 (I'm not against this change, just for information). 2018-03-29 16:05 GMT+03:00 Uros Bizjak

Re: [PATCH] i386: Enable AVX/AVX512 features only if supported by OSXSAVE

2018-03-29 Thread Uros Bizjak
On Thu, Mar 29, 2018 at 2:43 PM, H.J. Lu wrote: > Enable AVX and AVX512 features only if their states are supported by > OSXSAVE. > > OK for trunk and release branches? > > > H.J. > --- > PR target/85100 > * config/i386/cpuinfo.c (XCR_XFEATURE_ENABLED_MASK):