[PATCH] D28489: [CodeGen] Move MacroFusion to the target
Matthias Braun via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 27 13:45:33 PST 2017
MatzeB added a comment.
In https://reviews.llvm.org/D28489#659194, @evandro wrote:
> @MatzeB,
>
> I just thought that it was convenient to control `MacroFusion` with a global option, `misched-fusion`, regardless of what the target prefers.
I don't think a global flag is worth adding a callback and extra functions to MachineScheduler. I don't think there is that much value in the flag to justify that esp. since you can do -mattr=-FeatureArithmeticBccFusion,-FeatureArithmeticCbzFusion as well.
Repository:
rL LLVM
https://reviews.llvm.org/D28489
More information about the llvm-commits
mailing list