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

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 14 11:22:01 PST 2017


vsk added inline comments.


================
Comment at: include/llvm/Analysis/ScalarEvolutionExpander.h:182
+
+    /// deprecated{Please use the overload which requires a DebugLoc.}
     PHINode *getOrInsertCanonicalInductionVariable(const Loop *L, Type *Ty);
----------------
aprantl wrote:
> shouldn't this be `\deprecated` with a backslash? and come before the declaration?
Ouch, I forgot the backslash. I'm not sure what you mean by 'come before the declaration' though - it looks like it's before the declaration already.


https://reviews.llvm.org/D39985





More information about the llvm-commits mailing list