[PATCH] D133430: [SLP] Unify main/alternate instruction logic for CmpInst instructions - draft

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:490-494
+/// \returns true if compare instruction \p CI has similar "look" and
+/// same predicate as \p BaseCI
+/// or if compare instruction \p CI would have similar "look"
+/// to BaseCI when its operands and predicate swapped.
+/// \returns false otherwise.
----------------
Format comment


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:4563-4566
+static bool isAlternateInstruction(const Instruction *I,
+                                   const Instruction *MainOp,
+                                   const Instruction *AltOp);
+
----------------
Move the comment for the function here too, usually the description is associated with the first declaration.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133430