[PATCH] D13417: [MachineCombiner] make slack optional in critical path cost calculation (PR25016)
Gerolf Hoflehner via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 6 10:30:47 PDT 2015
Gerolf added a comment.
We should avoid scenarios where two types of machine combines suddenly would require a different heuristic within the same block, eg assume a madd combine and a reassoc in the same block. Also, on a minor note I don’t think I truly understand your underpinning performance issue yet.
Gerolf
http://reviews.llvm.org/D13417
More information about the llvm-commits
mailing list