[PATCH] [LoopAccesses] Stash the report from the analysis rather than emitting it

hfinkel at anl.gov hfinkel at anl.gov
Mon Feb 16 16:21:23 PST 2015


I imagine that, in the future, we might generate multiple report messages. I think that this is fine for now, however. LGTM.


================
Comment at: lib/Analysis/LoopAccessAnalysis.cpp:1080
@@ -1079,2 +1079,3 @@
 void LoopAccessInfo::emitAnalysis(VectorizationReport &Message) {
-  VectorizationReport::emitAnalysis(Message, TheFunction, TheLoop);
+  assert(!Report && "Multiple report generated");
+  Report = Message;
----------------
report -> reports

http://reviews.llvm.org/D7681

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






More information about the llvm-commits mailing list