On Thu, May 29, 2025 at 04:42:19PM +0100, Bruce Richardson wrote:
> When doing a build for a target that already has the instruction sets
> for AVX2/AVX512 enabled, skip emitting the AVX compiler flags, or the
> skylake-avx512 '-march' flags, as they are unnecessary. Instead, when
> the default flags produce the desired output, just use them unmodified,
> and don't bother adding in extra enabling flags for AVX2 or AVX-512.
>
> Depends-on: series-35006 ("doc/linux_gsg: update recommended compiler
> versions")
>
> Signed-off-by: Bruce Richardson <bruce.richard...@intel.com>
> ---
>
> V4: Fix error flagged by CI with clang builds without AVX512 - change
> "cc_avx512_args" to correct "cc_avx512_flags"
>
> V3: put in version check to work around an issues with some meson
> versions, (hopefully) allowing builds to pass in all CIs. The
> printout of the extra flags now only happens with meson >= 0.60.2
>
> V2: dropped the doc update for the minimum compiler version. Based on
> discussion, that version bump is larger than proposed in RFC and is
> now a separate patch/series [series 35006 referenced above]
>
> ---
> config/x86/meson.build | 31 ++++++++++++++++++++-----------
> drivers/meson.build | 9 +--------
> lib/meson.build | 9 +--------
> 3 files changed, 22 insertions(+), 27 deletions(-)
>
Recheck-request: github-robot