[PATCH] D36401: TTI: Use a better default for areInlineCompatibl

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 15 11:14:08 PDT 2017


MatzeB requested changes to this revision.
MatzeB added a comment.
This revision now requires changes to proceed.

I also think this patch is too optimistic. Not comparing the CPU name/feature attributes seems daring but doable to me. But I assuming all features are additive so we just need to check whether the callees flags are a subset of the callers flags seems too optimistic to me.


https://reviews.llvm.org/D36401





More information about the llvm-commits mailing list