[PATCH] D80012: [CostModel] Unify Intrinsic Costs.
Sam Parker via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 21 04:50:18 PDT 2020
samparker added a comment.
I'm suspecting that the debug changes maybe caused by the base implementation treating the debug intrinsics as free. I'll revert this and break it down into three separate patches:
1. Sink all the trivially free intrinsics into the bottom-most implementation.
2. Combine getIntrinsicCost and getIntrinsicInstrCost.
3. Have getInstructionThroughput use getUserCost.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80012/new/
https://reviews.llvm.org/D80012
More information about the llvm-commits
mailing list