[PATCH] TLI: Add addVectorizableFunctionsFromVecLib.

hfinkel at anl.gov hfinkel at anl.gov
Sun Mar 8 14:43:19 PDT 2015


================
Comment at: include/llvm/Analysis/TargetLibraryInfo.h:74
@@ -73,1 +73,3 @@
 public:
+  enum VectorLibrary {
+    NoLibrary, // Don't use any vector library.
----------------
Please add a comment here explaining what this general feature is, and that it can be used via the addVectorizableFunctionsFromVecLib function.


================
Comment at: include/llvm/Analysis/TargetLibraryInfo.h:127
@@ +126,3 @@
+  /// given vector library.
+  void addVectorizableFunctionsFromVecLib(enum VectorLibrary VecLib) {
+    switch (VecLib) {
----------------
Please don't put this implementation in the header. It will become large quickly.

http://reviews.llvm.org/D8131

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list