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

Adam Nemet anemet at apple.com
Mon Feb 16 16:25:59 PST 2015


In http://reviews.llvm.org/D7681#124563, @hfinkel wrote:

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


Yeah, initially I wrote this with a SmallVector.  When later in the patchset I was writing the part to prepend "loop cannot be vectorized" to the report for the vectorizer I realized that there was no point making that a loop, etc since we only have reports with a single message in them.


http://reviews.llvm.org/D7681

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






More information about the llvm-commits mailing list