[PATCH] D56530: [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 cfe-commits cfe-commits at lists.llvm.org
Wed Jan 9 18:34:27 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/D56530/new/

https://reviews.llvm.org/D56530





More information about the cfe-commits mailing list