[PATCH] D83111: [X86-64] Support Intel AMX Intrinsic

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 6 18:35:40 PDT 2020


craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.

LGTM with all instances of "pointer point" replace with just "pointer"



================
Comment at: clang/lib/Headers/amxintrin.h:33
+/// \param __config
+///    A pointer point to 512-bits configuration
+static __inline__ void __DEFAULT_FN_ATTRS
----------------
Drop 'point' here. I think its understood that a pointer points. And it would need to be "pointing" if it stays


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

https://reviews.llvm.org/D83111





More information about the llvm-commits mailing list