[PATCH] D139140: [ModuleInliner] Add MLPriority

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 1 14:40:20 PST 2022


mtrofin added a comment.

can you also add a test? thanks!



================
Comment at: llvm/lib/Analysis/InlineOrder.cpp:197
+private:
+  int Cost;
+};
----------------
can you initialize it here - lowers the risk of undefined values later (like if somehow the ctor forgets to initialize it)




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139140



More information about the llvm-commits mailing list