[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