[PATCH] D89969: [SLP] Consider alternatives for cost of select instructions.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 26 10:21:10 PDT 2020


RKSimon accepted this revision.
RKSimon added a comment.

LGTM with one minor



================
Comment at: llvm/lib/Analysis/ValueTracking.cpp:5991
+  // instructions is not done by SLPVectorizer and needs to be done by the
+  // backend.
+  // TODO: Support floating point min/max.
----------------
Update the comment?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89969



More information about the llvm-commits mailing list