[PATCH] D84167: [Attributor] Use internalized version of non-exact functions

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 20 06:27:36 PDT 2020


fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:1449
+  // Copy the body of the original function to the new one
+  llvm::CloneFunctionInto(Copied, &F, VMap, /* ModuleLevelChanges */ true,
+                          Returns);
----------------
nit: we are already using the llvm namespace, there should be no need to use `llvm::`. same at other places in the patch,


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84167





More information about the llvm-commits mailing list