[PATCH] D70107: [VFABI] TargetLibraryInfo mappings in IR.

[Francesco Petrogalli via Phabricator via llvm-commits]

fpetrogalli marked 2 inline comments as done.
fpetrogalli added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/InjectTLIMappings.cpp:28
+STATISTIC(NumVFDefAdded,
+          "Number of function definitions/declarations that have been added.");
+STATISTIC(NumCompUsedAdded,
----------------
TODO: The description of the counter is misleading, only declarations are added in this pass.


================
Comment at: llvm/lib/Transforms/Utils/InjectTLIMappings.cpp:139-142
+    for (auto &I : BB) {
+      if (auto CI = dyn_cast<CallInst>(&I))
+        addMappingsFromTLI(TLI, *CI);
+    }
----------------
TODO: remove braces.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70107