[PATCH] D35229: [CodeGen] Add support for instruction clusters

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 19 16:32:54 PDT 2017


MatzeB added a comment.

I'm not sure I like this:

- It introduces a new word "cluster", so we end up having the words "MacroFusion" and "Cluster" around which appear to mean the same thing?
- Using `computeOperandLatency()` to compute something called cluster feels out of place, similar with getReadAdvanceCycles


https://reviews.llvm.org/D35229





More information about the llvm-commits mailing list