[PATCH] D125712: [SLP][X86] Improve reordering to consider alternate instruction bundles

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 26 02:35:38 PDT 2022


RKSimon added inline comments.


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfo.h:684
+  bool isLegalAltInstr(VectorType *VecTy, unsigned EvenOpcode,
+                       unsigned OddOpcode) const;
+
----------------
If we're going to do this (and I'm still not convinced we should) we're going to need a shuffle mask - otherwise addsub matching is not going to be accurate.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125712/new/

https://reviews.llvm.org/D125712



More information about the llvm-commits mailing list