[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