russell.gallop created this revision. russell.gallop added reviewers: craig.topper, cfe-commits. russell.gallop added a project: clang.
Some of these test cases were using a variable instead of a literal so were not generating the immediate form of the corresponding instruction. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D62850 Files: clang/test/CodeGen/builtins-x86.c Index: clang/test/CodeGen/builtins-x86.c =================================================================== --- clang/test/CodeGen/builtins-x86.c +++ clang/test/CodeGen/builtins-x86.c @@ -399,14 +399,14 @@ #ifndef OPENCL (void) _mm_pause(); #endif - tmp_V4s = __builtin_ia32_psllwi(tmp_V4s, tmp_i); - tmp_V2i = __builtin_ia32_pslldi(tmp_V2i, tmp_i); - tmp_V1LLi = __builtin_ia32_psllqi(tmp_V1LLi, tmp_i); - tmp_V4s = __builtin_ia32_psrawi(tmp_V4s, tmp_i); - tmp_V2i = __builtin_ia32_psradi(tmp_V2i, tmp_i); - tmp_V4s = __builtin_ia32_psrlwi(tmp_V4s, tmp_i); - tmp_V2i = __builtin_ia32_psrldi(tmp_V2i, tmp_i); - tmp_V1LLi = __builtin_ia32_psrlqi(tmp_V1LLi, tmp_i); + tmp_V4s = __builtin_ia32_psllwi(tmp_V4s, imm_i_0_8); + tmp_V2i = __builtin_ia32_pslldi(tmp_V2i, imm_i_0_8); + tmp_V1LLi = __builtin_ia32_psllqi(tmp_V1LLi, imm_i_0_8); + tmp_V4s = __builtin_ia32_psrawi(tmp_V4s, imm_i_0_8); + tmp_V2i = __builtin_ia32_psradi(tmp_V2i, imm_i_0_8); + tmp_V4s = __builtin_ia32_psrlwi(tmp_V4s, imm_i_0_8); + tmp_V2i = __builtin_ia32_psrldi(tmp_V2i, imm_i_0_8); + tmp_V1LLi = __builtin_ia32_psrlqi(tmp_V1LLi, imm_i_0_8); tmp_V1LLi = __builtin_ia32_pmuludq(tmp_V2i, tmp_V2i); tmp_V2LLi = __builtin_ia32_pmuludq128(tmp_V4i, tmp_V4i); tmp_V8s = __builtin_ia32_psraw128(tmp_V8s, tmp_V8s); @@ -417,14 +417,14 @@ tmp_V8s = __builtin_ia32_psllw128(tmp_V8s, tmp_V8s); tmp_V4i = __builtin_ia32_pslld128(tmp_V4i, tmp_V4i); tmp_V2LLi = __builtin_ia32_psllq128(tmp_V2LLi, tmp_V2LLi); - tmp_V8s = __builtin_ia32_psllwi128(tmp_V8s, tmp_i); - tmp_V4i = __builtin_ia32_pslldi128(tmp_V4i, tmp_i); - tmp_V2LLi = __builtin_ia32_psllqi128(tmp_V2LLi, tmp_i); - tmp_V8s = __builtin_ia32_psrlwi128(tmp_V8s, tmp_i); - tmp_V4i = __builtin_ia32_psrldi128(tmp_V4i, tmp_i); - tmp_V2LLi = __builtin_ia32_psrlqi128(tmp_V2LLi, tmp_i); - tmp_V8s = __builtin_ia32_psrawi128(tmp_V8s, tmp_i); - tmp_V4i = __builtin_ia32_psradi128(tmp_V4i, tmp_i); + tmp_V8s = __builtin_ia32_psllwi128(tmp_V8s, imm_i_0_8); + tmp_V4i = __builtin_ia32_pslldi128(tmp_V4i, imm_i_0_8); + tmp_V2LLi = __builtin_ia32_psllqi128(tmp_V2LLi, imm_i_0_8); + tmp_V8s = __builtin_ia32_psrlwi128(tmp_V8s, imm_i_0_8); + tmp_V4i = __builtin_ia32_psrldi128(tmp_V4i, imm_i_0_8); + tmp_V2LLi = __builtin_ia32_psrlqi128(tmp_V2LLi, imm_i_0_8); + tmp_V8s = __builtin_ia32_psrawi128(tmp_V8s, imm_i_0_8); + tmp_V4i = __builtin_ia32_psradi128(tmp_V4i, imm_i_0_8); tmp_V4i = __builtin_ia32_pmaddwd128(tmp_V8s, tmp_V8s); (void) __builtin_ia32_monitor(tmp_vp, tmp_Ui, tmp_Ui); (void) __builtin_ia32_mwait(tmp_Ui, tmp_Ui);
Index: clang/test/CodeGen/builtins-x86.c =================================================================== --- clang/test/CodeGen/builtins-x86.c +++ clang/test/CodeGen/builtins-x86.c @@ -399,14 +399,14 @@ #ifndef OPENCL (void) _mm_pause(); #endif - tmp_V4s = __builtin_ia32_psllwi(tmp_V4s, tmp_i); - tmp_V2i = __builtin_ia32_pslldi(tmp_V2i, tmp_i); - tmp_V1LLi = __builtin_ia32_psllqi(tmp_V1LLi, tmp_i); - tmp_V4s = __builtin_ia32_psrawi(tmp_V4s, tmp_i); - tmp_V2i = __builtin_ia32_psradi(tmp_V2i, tmp_i); - tmp_V4s = __builtin_ia32_psrlwi(tmp_V4s, tmp_i); - tmp_V2i = __builtin_ia32_psrldi(tmp_V2i, tmp_i); - tmp_V1LLi = __builtin_ia32_psrlqi(tmp_V1LLi, tmp_i); + tmp_V4s = __builtin_ia32_psllwi(tmp_V4s, imm_i_0_8); + tmp_V2i = __builtin_ia32_pslldi(tmp_V2i, imm_i_0_8); + tmp_V1LLi = __builtin_ia32_psllqi(tmp_V1LLi, imm_i_0_8); + tmp_V4s = __builtin_ia32_psrawi(tmp_V4s, imm_i_0_8); + tmp_V2i = __builtin_ia32_psradi(tmp_V2i, imm_i_0_8); + tmp_V4s = __builtin_ia32_psrlwi(tmp_V4s, imm_i_0_8); + tmp_V2i = __builtin_ia32_psrldi(tmp_V2i, imm_i_0_8); + tmp_V1LLi = __builtin_ia32_psrlqi(tmp_V1LLi, imm_i_0_8); tmp_V1LLi = __builtin_ia32_pmuludq(tmp_V2i, tmp_V2i); tmp_V2LLi = __builtin_ia32_pmuludq128(tmp_V4i, tmp_V4i); tmp_V8s = __builtin_ia32_psraw128(tmp_V8s, tmp_V8s); @@ -417,14 +417,14 @@ tmp_V8s = __builtin_ia32_psllw128(tmp_V8s, tmp_V8s); tmp_V4i = __builtin_ia32_pslld128(tmp_V4i, tmp_V4i); tmp_V2LLi = __builtin_ia32_psllq128(tmp_V2LLi, tmp_V2LLi); - tmp_V8s = __builtin_ia32_psllwi128(tmp_V8s, tmp_i); - tmp_V4i = __builtin_ia32_pslldi128(tmp_V4i, tmp_i); - tmp_V2LLi = __builtin_ia32_psllqi128(tmp_V2LLi, tmp_i); - tmp_V8s = __builtin_ia32_psrlwi128(tmp_V8s, tmp_i); - tmp_V4i = __builtin_ia32_psrldi128(tmp_V4i, tmp_i); - tmp_V2LLi = __builtin_ia32_psrlqi128(tmp_V2LLi, tmp_i); - tmp_V8s = __builtin_ia32_psrawi128(tmp_V8s, tmp_i); - tmp_V4i = __builtin_ia32_psradi128(tmp_V4i, tmp_i); + tmp_V8s = __builtin_ia32_psllwi128(tmp_V8s, imm_i_0_8); + tmp_V4i = __builtin_ia32_pslldi128(tmp_V4i, imm_i_0_8); + tmp_V2LLi = __builtin_ia32_psllqi128(tmp_V2LLi, imm_i_0_8); + tmp_V8s = __builtin_ia32_psrlwi128(tmp_V8s, imm_i_0_8); + tmp_V4i = __builtin_ia32_psrldi128(tmp_V4i, imm_i_0_8); + tmp_V2LLi = __builtin_ia32_psrlqi128(tmp_V2LLi, imm_i_0_8); + tmp_V8s = __builtin_ia32_psrawi128(tmp_V8s, imm_i_0_8); + tmp_V4i = __builtin_ia32_psradi128(tmp_V4i, imm_i_0_8); tmp_V4i = __builtin_ia32_pmaddwd128(tmp_V8s, tmp_V8s); (void) __builtin_ia32_monitor(tmp_vp, tmp_Ui, tmp_Ui); (void) __builtin_ia32_mwait(tmp_Ui, tmp_Ui);
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
