[PATCH] D28476: [Analysis] Add LF_ prefix to LibFunc enums in TargetLibraryInfo. (NFC)

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 23 15:21:07 PST 2017


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

LGTM with a couple minor tweaks.  (If you don't have commit access, upload an updated version and ping me.)



================
Comment at: lib/Transforms/Scalar/DeadStoreElimination.cpp:147
+      }
+      if (TLI.has(LibFunc_strncat) && FnName == TLI.getName(LibFunc_strncat)) {
         return true;
----------------
Not sure why you're changing this.


================
Comment at: lib/Transforms/Scalar/MemCpyOptimizer.cpp:1277
 
-  if (!TLI->has(LibFunc::memmove))
+  if (!TLI->has(LibFunc::LibFunc_memmove))
     return false;
----------------
?


https://reviews.llvm.org/D28476





More information about the llvm-commits mailing list