[PATCH] D28693: [InlineCost] Use TTI to check if GEP is free.
Chad Rosier via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 18 11:02:18 PST 2017
mcrosier added a comment.
LGTM, but this should be further reviewed by someone with more familiarity of the inliner cost model.
================
Comment at: test/Transforms/Inline/gep-cost.ll:1
+; RUN: opt -inline < %s -S -debug-only=inline-cost 2>&1 | FileCheck %s
+
----------------
I apologize for my lack of understanding. Assuming this isn't obvious to someone more familiar with the inliner cost model, would you mind adding a few comments to this test case explaining exactly what you're testing?
I diffed the output from this test with and without your change and the only cost that was changed was for inner1. Thus, I'm trying to understand the significance of inner2.
Repository:
rL LLVM
https://reviews.llvm.org/D28693
More information about the llvm-commits
mailing list