[PATCH] [LoopInterchange] Add support to interchange loops with reductions.

Renato Golin renato.golin at linaro.org
Wed Apr 22 09:53:18 PDT 2015


Hi Karthik,

Looks good to me with the two nitpicks. Feel free to commit with those changes.

Thanks!
-renato


================
Comment at: test/Transforms/LoopInterchange/reductions.ll:49
@@ +48,3 @@
+; CHECK:    br label %for.body3.lr.ph.preheader
+; CHECK:    %add = add nsw i32 %X.promoted, %0
+
----------------
nitpick: you don't need to match %0 here. It won't match if some pass adds a new unrelated operation. Just match up to promoted.

================
Comment at: test/Transforms/LoopInterchange/reductions.ll:113
@@ +112,3 @@
+; CHECK:    br label %for.cond4.preheader.preheader.preheader
+; CHECK:    %add13 = add nsw i32 %Y.promoted, %1
+
----------------
Same here. Mainly because %1 and %0 were not part of the rest of the match.

http://reviews.llvm.org/D8314

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list