[PATCH] D35758: [LIR] Teach LIR to avoid extending the BE count prior to adding one to it when safe.

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 21 20:32:15 PDT 2017


chandlerc added a comment.

Wheee!



================
Comment at: lib/Transforms/Scalar/LoopIdiomRecognize.cpp:790
+                               const DataLayout *DL, ScalarEvolution *SE) {
+  const SCEV *NumBytesS;
+  // The # stored bytes is (BECount+1)*Size.  Expand the trip count out to
----------------
sanjoy wrote:
> Not sure what the `S` at the end of `NumBytes` is for.
It was there in the original code. I'm guessing it means "SCEV"? Anyways, happy to nuke it if you want (in a subsequent patch).


https://reviews.llvm.org/D35758





More information about the llvm-commits mailing list