[PATCH] D33203: Add scheduler classes to integer/float horizontal operations

Andrew V. Tischenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 1 05:43:21 PDT 2017


avt77 added inline comments.


================
Comment at: lib/Target/X86/X86Schedule.td:82
+defm WriteHAdd  : X86SchedWritePair;
+defm WriteHAddY : X86SchedWritePair;
+defm WritePHAdd : X86SchedWritePair;
----------------
RKSimon wrote:
> I'm not sure we should be introducing size specific versions - we don't do this for any other cases. Merge back into WriteHAdd?
I did it specially because Jaguar has different numbers for Ymm and Xmm and we can't model such difference without the special SchedWrite. Is it OK?


https://reviews.llvm.org/D33203





More information about the llvm-commits mailing list