[PATCH] D39985: [SCEV] Apply a single debug loc when expanding a SCEV

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 13 18:50:26 PST 2017


aprantl added inline comments.


================
Comment at: include/llvm/Analysis/ScalarEvolutionExpander.h:180
+    PHINode *getOrInsertCanonicalInductionVariable(const Loop *L,
+                                                   const DebugLoc &DbgLoc,
+                                                   Type *Ty);
----------------
I think DebugLoc is already a value type, so there is no point in passing it by reference.


https://reviews.llvm.org/D39985





More information about the llvm-commits mailing list