diff options
| author | Victor Umansky <victor.umansky@intel.com> | 2012-01-26 08:51:39 +0000 |
|---|---|---|
| committer | Victor Umansky <victor.umansky@intel.com> | 2012-01-26 08:51:39 +0000 |
| commit | 668f7ac9e4642e1b4af4f5b047d569c68bc2c55f (patch) | |
| tree | e2865732f4a8dc4baeb255150480f00513e51281 /test/CodeGen/X86/avx-cvt.ll | |
| parent | a3937416e43b485633035e8184755d9a94efb7b3 (diff) | |
| download | llvm-668f7ac9e4642e1b4af4f5b047d569c68bc2c55f.tar.gz llvm-668f7ac9e4642e1b4af4f5b047d569c68bc2c55f.tar.bz2 llvm-668f7ac9e4642e1b4af4f5b047d569c68bc2c55f.tar.xz | |
Fix for the following bug in AVX codegen for double-to-int conversions:
. "fptosi" and "fptoui" IR instructions are defined with round-to-zero rounding mode.
. Currently for AVX mode for <4xdouble> and <8xdouble> the "VCVTPD2DQ.128" and "VCVTPD2DQ.256" instructions are selected (for .fp_to_sint. DAG node operation ) by AVX codegen. However they use round-to-nearest-even rounding mode.
. Consequently, the conversion produces incorrect numbers.
The fix is to replace selection of VCVTPD2DQ instructions with VCVTTPD2DQ instructions. The latter use truncate (i.e. round-to-zero) rounding mode.
As .fp_to_sint. DAG node operation is used only for lowering of "fptosi" and "fptoui" IR instructions, the fix in X86InstrSSE.td definition file doesn.t have an impact on other LLVM flows.
The patch includes changes in the .td file, LIT test for the changes and a fix in a legacy LIT test (which produced asm code conflicting with LLVN IR spec).
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@149056 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/CodeGen/X86/avx-cvt.ll')
| -rw-r--r-- | test/CodeGen/X86/avx-cvt.ll | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/CodeGen/X86/avx-cvt.ll b/test/CodeGen/X86/avx-cvt.ll index 6c0bd58074..d0a7fe0100 100644 --- a/test/CodeGen/X86/avx-cvt.ll +++ b/test/CodeGen/X86/avx-cvt.ll @@ -18,7 +18,7 @@ define <4 x double> @sitofp01(<4 x i32> %a) { ret <4 x double> %b } -; CHECK: vcvtpd2dqy %ymm +; CHECK: vcvttpd2dqy %ymm define <4 x i32> @fptosi01(<4 x double> %a) { %b = fptosi <4 x double> %a to <4 x i32> ret <4 x i32> %b |