[PATCH] D34057: [TargetTransformInfo, API] Add optional list of operands to TTI::getUserCost

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 20 15:27:18 PDT 2017


efriedma added a comment.

Yes, this is what I was meant; thanks.



================
Comment at: include/llvm/Analysis/TargetTransformInfo.h:234
+                                           U->value_op_end());
+    return getUserCost(U, std::move(Operands));
+  }
----------------
std::move doesn't do anything here; ArrayRef is POD.


https://reviews.llvm.org/D34057





More information about the llvm-commits mailing list