[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-05-03 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 H.J. Lu changed: What|Removed |Added CC||luto at kernel dot org --- Comment #13 from

[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-04-02 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 H.J. Lu changed: What|Removed |Added Status|WAITING |RESOLVED Resolution|---

[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-04-02 Thread hjl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 --- Comment #11 from hjl at gcc dot gnu.org --- Author: hjl Date: Mon Apr 2 12:09:48 2018 New Revision: 259007 URL: https://gcc.gnu.org/viewcvs?rev=259007=gcc=rev Log: i386: Enable AVX/AVX512 features only if supported by OSXSAVE Enable AVX

[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-04-02 Thread hjl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 --- Comment #10 from hjl at gcc dot gnu.org --- Author: hjl Date: Mon Apr 2 12:03:16 2018 New Revision: 259006 URL: https://gcc.gnu.org/viewcvs?rev=259006=gcc=rev Log: i386: Enable AVX/AVX512 features only if supported by OSXSAVE Enable AVX

[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-03-29 Thread hjl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 --- Comment #9 from hjl at gcc dot gnu.org --- Author: hjl Date: Thu Mar 29 13:14:06 2018 New Revision: 258954 URL: https://gcc.gnu.org/viewcvs?rev=258954=gcc=rev Log: i386: Enable AVX/AVX512 features only if supported by OSXSAVE Enable AVX

[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-03-29 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 --- Comment #8 from Uroš Bizjak --- Comment on attachment 43793 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43793 A patch >+ if ((ecx & bit_OSXSAVE)) >+{ >+ /* Check if XMM state and YMM state are saved. */ >+ unsigned

[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-03-29 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 Uroš Bizjak changed: What|Removed |Added Target Milestone|--- |6.5 Severity|enhancement

[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-03-29 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 --- Comment #6 from Uroš Bizjak --- (In reply to Richard Biener from comment #2) > I think this is on purpose and unlikely to change. No, __builtin_cpu_supports is used to enable parts that are able to execute relevant instructions. So, HJ's

[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-03-28 Thread njs at pobox dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 --- Comment #5 from Nathaniel J. Smith --- Julian, are you able to test the patch? I don't have a reproduction setup currently...

[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-03-28 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 H.J. Lu changed: What|Removed |Added Status|NEW |WAITING

[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-03-28 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 H.J. Lu changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-03-28 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 H.J. Lu changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |hjl.tools at gmail dot com ---

[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-03-28 Thread njs at pobox dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 --- Comment #3 from Nathaniel J. Smith --- We're using it exactly like the docs recommend. What on earth is __builtin_cpu_supports for, if not to tell you whether you can use a given feature? If this is by design, than at the very least the

[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-03-28 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 Richard Biener changed: What|Removed |Added Target||x86_64-*-*, i?86-*-*

[Bug target/85100] __builtin_cpu_supports avx does not verify OS supports it

2018-03-27 Thread njs at pobox dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85100 Nathaniel J. Smith changed: What|Removed |Added CC||njs at pobox dot com --- Comment