[PATCH] D80186: [Inliner] Update !associated metadata during inlining

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 19 16:34:43 PDT 2020


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

LGTM



================
Comment at: llvm/lib/Transforms/Utils/InlineFunction.cpp:1148
+
+  for (GlobalVariable &GV : M->globals()) {
+    MDNode *MD = GV.getMetadata(LLVMContext::MD_associated);
----------------
This is technically N^2 if you do this for every function. I guess it's fine as long as you don't see a bump in the compilation time.


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

https://reviews.llvm.org/D80186





More information about the llvm-commits mailing list