<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jul 3, 2013 at 11:38 AM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 07/02/2013 06:34 PM, Hongbin Zheng wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
Hi tobi,<br>
<br>
<br>
On Wed, Jul 3, 2013 at 12:21 AM, Tobias Grosser <<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>> wrote:<br>
<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
On 06/29/2013 12:00 AM, Hongbin Zheng wrote:<br>
<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
Author: ether<br>
Date: Sat Jun 29 02:00:14 2013<br>
New Revision: 185254<br>
<br></div>
URL: <a href="http://llvm.org/viewvc/llvm-**project?rev=185254&view=rev" target="_blank">http://llvm.org/viewvc/llvm-**<u></u>project?rev=185254&view=rev</a><<a href="http://llvm.org/viewvc/llvm-project?rev=185254&view=rev" target="_blank">ht<u></u>tp://llvm.org/viewvc/llvm-<u></u>project?rev=185254&view=rev</a>><div class="im">
<br>
Log:<br>
TempScop: (Partial) Implement the printDetail function.<br>
<br>
</div></blockquote><div class="im">
<br>
Thanks!<br>
<br>
Also, it would be nice to check the changes with clang-format before you<br>
commit them.<br>
<br>
If clang-format is in your path, you can just run 'make<br>
polly-check-format' (or 'make polly-update-format' to fix the formatting)<br>
<br>
Some comments inline.<br>
<br>
Cheers<br>
Tobias<br>
<br>
<br></div>
    void TempScop::printDetail(raw_**<u></u>ostream &OS, ScalarEvolution *SE,<div class="im"><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
                              LoopInfo *LI, const Region *CurR,<br>
-                           unsigned ind) const {}<br>
+                           unsigned ind) const {<br>
+<br>
+  // FIXME: Print other details rather than memory accesses.<br>
+  typedef Region::const_block_iterator bb_iterator;<br>
+  for (bb_iterator I = CurR->block_begin(), E = CurR->block_end(); I !=<br>
E; ++I){<br>
+    BasicBlock *CurBlock = *I;<br>
+<br>
+    AccFuncMapType::const_iterator AccSetIt = AccFuncMap.find(CurBlock);<br>
+<br>
+    // Ignore trivial blocks that do not contain any memory access.<br>
+    if (AccSetIt == AccFuncMap.end()) continue;<br>
<br>
</blockquote>
<br>
It would be nice to move this into a separate function that is also used<br>
in place of the Scop::isTrivialBB() function in ScopInfo.cpp.<br>
<br>
</div></blockquote><div class="im">
Ok<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Such a patch is approved for commit. ;-)<br>
</blockquote>
<br>
Well, this patch is reviewed before, you said once I shorten the commit<br>
message, it is ok to commit ;-)<br>
</div></blockquote>
<br>
Yes, I remember. I took a too long delay in reviewing your patches, so I forgot it for a moment.<br></blockquote><div>Ok, I will be careful next time </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
Cheers,<br>
Tobi<br>
</blockquote></div><br></div></div>