[PATCH] D104028: [llvm][Inliner] Add an optional PriorityInlineOrder

[Mircea Trofin via Phabricator via llvm-commits]

mtrofin added inline comments.


================
Comment at: llvm/test/Transforms/Inline/monster_scc.ll:73
 ; NEW-NOT: call
+; PO-NOT: call
+; PO: call void @_Z1fILb1ELi1EEvPbS0_(
----------------
These look like the old cases, though - is there a difference that the priority-based ordering introduces? (maybe I'm missing it)


================
Comment at: llvm/test/Transforms/Inline/monster_scc.ll:132
 ; NEW-NOT: call
+; PO-NOT: call
+; PO: call void @_Z1gi(
----------------
same comment - what's the difference?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104028