[PATCH] D28489: [CodeGen] Generalize MacroFusion for any instructions pair
Andrew Trick via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 18 11:40:24 PST 2017
atrick added a comment.
I think you should find a way to do this without calling shouldScheduleAdjacent on every DAG node. It's fine to say you've tested compilation time but what really matters here are the pathological cases with very large blocks. It's rare for instructions in the middle of blocks to have fusion opportunities, so it's wrong to introduce this potential cost for all blocks.
More information about the llvm-commits