[PATCH] D93594: [X86] Pass to transform amx intrinsics to scalar operation.

LuoYuanke via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Feb 27 04:49:52 PST 2021


LuoYuanke added inline comments.


================
Comment at: llvm/lib/Target/X86/X86LowerAMXIntrinsics.cpp:88
+
+template <Intrinsic::ID IntrID,
+          typename = typename std::enable_if<
----------------
pengfei wrote:
> Can we just use `template <bool IsLoad>`? I think it also can reduce the branch.
Why do we need a template instead of passing a parameter `bool IsLoad`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93594



More information about the cfe-commits mailing list