[PATCH] D42983: [clang-tidy] Add readability-simd-intrinsics check.

Tim Shen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 6 18:23:24 PST 2018


timshen added inline comments.


================
Comment at: clang-tidy/readability/SIMDIntrinsicsCheck.cpp:22
+
+static StringRef CheckPPC(StringRef Name) {
+  if (Name.startswith("vec_"))
----------------
"Check" usually indicates to return a bool, but what it actually returns is a possible mapped result based on the input. Maybe some name like "TrySuggestPPC"?


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D42983





More information about the cfe-commits mailing list