[PATCH] D32943: [SCEV] Use move semantics in ScalarEvolution::setRange

Takuto Ikuta via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 6 18:11:43 PDT 2017


takuto.ikuta added inline comments.


================
Comment at: include/llvm/Analysis/ScalarEvolution.h:785
   const ConstantRange &setRange(const SCEV *S, RangeSignHint Hint,
-                                const ConstantRange &CR) {
+                                ConstantRange CR) {
     DenseMap<const SCEV *, ConstantRange> &Cache =
----------------
Better to change type to `ConstantRange&&`?


https://reviews.llvm.org/D32943





More information about the llvm-commits mailing list