From: Paolo Bonzini <pbonzini@redhat.com> To: qemu-devel@nongnu.org Cc: Richard Henderson <richard.henderson@linaro.org> Subject: [PULL 3/7] configure: Add -Werror to avx2, avx512 tests Date: Thu, 29 Jul 2021 14:47:09 +0200 [thread overview] Message-ID: <20210729124713.208422-4-pbonzini@redhat.com> (raw) In-Reply-To: <20210729124713.208422-1-pbonzini@redhat.com> From: Richard Henderson <richard.henderson@linaro.org> When using clang, we get ERROR: configure test passed without -Werror but failed with -Werror. This is probably a bug in the configure script. The failing command will be at the bottom of config.log. You can run configure with --disable-werror to bypass this check. What we really want from these two tests is whether the entire code sequence is supported, including pragmas. Adding -Werror makes the test properly fail for clang. Signed-off-by: Richard Henderson <richard.henderson@linaro.org> Message-Id: <20210719200112.295316-1-richard.henderson@linaro.org> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> --- configure | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 79e2ddc74e..422a456f0b 100755 --- a/configure +++ b/configure @@ -3881,7 +3881,7 @@ static int bar(void *a) { } int main(int argc, char *argv[]) { return bar(argv[0]); } EOF - if compile_object "" ; then + if compile_object "-Werror" ; then avx2_opt="yes" else avx2_opt="no" @@ -3911,7 +3911,7 @@ int main(int argc, char *argv[]) return bar(argv[0]); } EOF - if ! compile_object "" ; then + if ! compile_object "-Werror" ; then avx512f_opt="no" fi else -- 2.31.1
next prev parent reply other threads:[~2021-07-29 12:53 UTC|newest] Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-07-29 12:47 [PULL 0/7] Misc patches for QEMU 6.1-rc2 Paolo Bonzini 2021-07-29 12:47 ` [PULL 1/7] i386: assert 'cs->kvm_state' is not null Paolo Bonzini 2021-07-29 12:47 ` [PULL 2/7] Makefile: ignore long options Paolo Bonzini 2021-07-29 12:47 ` Paolo Bonzini [this message] 2021-07-29 12:47 ` [PULL 4/7] target/i386: Added consistency checks for event injection Paolo Bonzini 2021-07-29 12:47 ` [PULL 5/7] target/i386: fix typo in ctl_has_irq Paolo Bonzini 2021-07-29 12:47 ` [PULL 6/7] meson: fix meson 0.58 warning with libvhost-user subproject Paolo Bonzini 2021-07-29 12:58 ` Peter Maydell 2021-07-29 14:05 ` Thomas Huth 2021-07-29 15:07 ` Peter Maydell 2021-07-29 16:50 ` 罗勇刚(Yonggang Luo) 2021-08-19 8:43 ` Christian Borntraeger 2021-08-19 14:51 ` Christian Borntraeger 2021-08-19 15:11 ` Christian Borntraeger 2021-07-29 12:47 ` [PULL 7/7] libvhost-user: fix -Werror=format= warnings with __u64 fields Paolo Bonzini 2021-07-29 17:48 ` [PULL 0/7] Misc patches for QEMU 6.1-rc2 Peter Maydell
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20210729124713.208422-4-pbonzini@redhat.com \ --to=pbonzini@redhat.com \ --cc=qemu-devel@nongnu.org \ --cc=richard.henderson@linaro.org \ --subject='Re: [PULL 3/7] configure: Add -Werror to avx2, avx512 tests' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.