Re: [PATCH] [PR91400] build only one __cpu_model variable

2021-05-03 Thread Ivan Sorokin via Gcc-patches
As GCC 11 is released and the development of GCC 12 begins I would like to send this patch again. It is basically unchanged since the previous submission in February. The only change I made is fixing formatting by adding a space after gcc_assert. Please note, that the original author of the co

[PATCH] [PR91400] build only one __cpu_model variable

2021-05-03 Thread Ivan Sorokin via Gcc-patches
Prior to this commit GCC -O2 generated quite bad code for this function: bool f() { return __builtin_cpu_supports("popcnt") && __builtin_cpu_supports("ssse3"); } f: movl__cpu_model+12(%rip), %eax xorl%r8d, %r8d testb $4, %al je .L1

Re: [PATCH] [PR91400] build only one __cpu_model variable

2021-02-01 Thread Ivan Sorokin via Gcc-patches
On 02.02.2021 02:00, Ivan Sorokin wrote: > [PATCH] [PR91400] build only one __cpu_model variable This is my first patch to GCC. So I might have done something totally stupid or totally wrong. Caution is required for reviewing. :-) > gcc/ChangeLog: > > PR target/91400 > * config/i386/

[PATCH] [PR91400] build only one __cpu_model variable

2021-02-01 Thread Ivan Sorokin via Gcc-patches
Prior to this commit GCC -O2 generated quite bad code for this function: bool f() { return __builtin_cpu_supports("popcnt") && __builtin_cpu_supports("ssse3"); } f: movl__cpu_model+12(%rip), %eax xorl%r8d, %r8d testb $4, %al je .L1