On Tue, Feb 19, 2019 at 9:14 PM H.J. Lu <hjl.to...@gmail.com> wrote: > > On Tue, Feb 19, 2019 at 6:16 AM Uros Bizjak <ubiz...@gmail.com> wrote: > > > > On Tue, Feb 19, 2019 at 2:49 PM H.J. Lu <hjl.to...@gmail.com> wrote: > > > > > > ix86_fpmath should be set to combination of FPMATH_387 and FPMATH_SSE. > > > When SSE is disabled, it should be set to FPMATH_387 and 387 codegen is > > > also controlled by -msoft-float. > > > > > > gcc/ > > > > > > PR target/89397 > > > * config/i386/i386.c (ix86_option_override_internal): Set > > > opts->x_ix86_fpmath to FPMATH_387 when SSE is disabled. > > > > > > gcc/testsuite/ > > > > > > PR target/89397 > > > * gcc.target/i386/pr89397.c: New test. > > > > OK. > > > > This patch is need to fix: > > FAIL: gcc.target/i386/pr67985-3.c scan-assembler movd[ \t]%xmm[0-7], %eax > FAIL: gcc.target/i386/pr67985-3.c scan-assembler mulss > > OK for trunk?
PR 67985 says: --q-- Since -miamcu specifies how parameters are passed to functions, inside function, we can use any instructions -march= allows. -miamcu -march=haswell should allow x87 and AVX instructions --/q-- The command line specifies -mfpmath=sse and target attribute implies SSE. So, SSE instructions should be used here. Please investigate why this is not the case. Uros. Uros.
