[PATCH] D56529: [X86] Add versions of the avx512 gather intrinsics that take the mask as a vXi1 vector instead of a scalar (clang side)

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 9 18:33:31 PST 2019


craig.topper marked an inline comment as done.
craig.topper added inline comments.


================
Comment at: include/clang/Basic/BuiltinsX86.def:988
 TARGET_BUILTIN(__builtin_ia32_gathersiv8df, "V8dV8dvC*V8iUcIi", "nV:512:", "avx512f")
-TARGET_BUILTIN(__builtin_ia32_gathersiv16sf, "V16fV16fvC*V16fUsIi", "nV:512:", "avx512f")
+TARGET_BUILTIN(__builtin_ia32_gathersiv16sf, "V16fV16fvC*V16iUsIi", "nV:512:", "avx512f")
 TARGET_BUILTIN(__builtin_ia32_gatherdiv8df, "V8dV8dvC*V8LLiUcIi", "nV:512:", "avx512f")
----------------
This was a bug that seems to have been hidden because generic builtin to intrinsic handling blindly inserted a bitcast to match the type. I'll probably pre-commit this


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56529/new/

https://reviews.llvm.org/D56529





More information about the llvm-commits mailing list