[PATCH] D81515: [llvm] Release-mode ML InlineAdvisor

[Mircea Trofin via Phabricator via llvm-commits]

mtrofin marked 4 inline comments as done.
mtrofin added inline comments.


================
Comment at: llvm/test/Transforms/Inline/ML/bounds-checks.ll:37
+; CHECK-NOT: call i64 @f2
+; NOBOUNDS-NOT: @some_function
+; BOUNDS: call i64 @some_function
----------------
davidxl wrote:
> can you explain about the expected output?
Added more detail


================
Comment at: llvm/test/Transforms/Inline/ML/ml-test-release-mode.ll:1
+; The default inliner doesn't elide @adder, the ML inliner does.
+;
----------------
davidxl wrote:
> Why can't default inliner handle this case (adder call can be folded). 
Cost evaluation - added explanation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81515