<div dir="ltr"><a href="http://llvm.org/PR14747">llvm.org/PR14747</a><br><div style><a href="http://llvm.org/PR15158">llvm.org/PR15158</a></div><div style><br></div><div style>The decision (mostly by Chris and Douglas) is to put the spaces for consistency.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Feb 5, 2013 at 7:38 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Feb 5, 2013 at 3:56 AM, Tobias Grosser<br>
<<a href="mailto:grosser@fim.uni-passau.de">grosser@fim.uni-passau.de</a>> wrote:<br>
> Author: grosser<br>
> Date: Tue Feb  5 05:56:05 2013<br>
> New Revision: 174365<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=174365&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=174365&view=rev</a><br>
> Log:<br>
> ScopDetect: clang-format pointer types in templates<br>
><br>
> Modified:<br>
>     polly/trunk/lib/Analysis/ScopDetection.cpp<br>
><br>
> Modified: polly/trunk/lib/Analysis/ScopDetection.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/polly/trunk/lib/Analysis/ScopDetection.cpp?rev=174365&r1=174364&r2=174365&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/polly/trunk/lib/Analysis/ScopDetection.cpp?rev=174365&r1=174364&r2=174365&view=diff</a><br>

> ==============================================================================<br>
> --- polly/trunk/lib/Analysis/ScopDetection.cpp (original)<br>
> +++ polly/trunk/lib/Analysis/ScopDetection.cpp Tue Feb  5 05:56:05 2013<br>
> @@ -281,14 +281,14 @@ bool ScopDetection::isValidMemoryAccess(<br>
><br>
>      OS << "Possible aliasing: ";<br>
><br>
> -    std::vector<Value*> Pointers;<br>
> +    std::vector<Value *> Pointers;<br>
<br>
Daniel - have you had any discussion on this particular issue? (do<br>
LLVM developers have any opinions, too)<br>
<br>
I've often considered skipping the space in contexts where there is no<br>
identifier to bind the '*' to on the right hand side (so - template<br>
arguments, unnamed parameter declarations, etc). Right/wrong/meh?<br>
<br>
><br>
>      for (AliasSet::iterator AI = AS.begin(), AE = AS.end(); AI != AE; ++AI)<br>
>        Pointers.push_back(AI.getPointer());<br>
><br>
>      std::sort(Pointers.begin(), Pointers.end());<br>
><br>
> -    for (std::vector<Value*>::iterator PI = Pointers.begin(),<br>
> +    for (std::vector<Value *>::iterator PI = Pointers.begin(),<br>
>                                          PE = Pointers.end();<br>
>           ;) {<br>
>        Value *V = *PI;<br>
> @@ -475,7 +475,7 @@ void ScopDetection::findScops(Region &R)<br>
>    // regions that form a Scop are not found. Therefore, those non canonical<br>
>    // regions are checked by expanding the canonical ones.<br>
><br>
> -  std::vector<Region*> ToExpand;<br>
> +  std::vector<Region *> ToExpand;<br>
><br>
>    for (Region::iterator I = R.begin(), E = R.end(); I != E; ++I)<br>
>      ToExpand.push_back(*I);<br>
> @@ -633,7 +633,7 @@ bool ScopDetection::runOnFunction(llvm::<br>
><br>
>  void polly::ScopDetection::verifyRegion(const Region &R) const {<br>
>    assert(isMaxRegionInScop(R) && "Expect R is a valid region.");<br>
> -  DetectionContext Context(const_cast<Region&>(R), *AA, true /*verifying*/);<br>
> +  DetectionContext Context(const_cast<Region &>(R), *AA, true /*verifying*/);<br>
>    isValidRegion(Context);<br>
>  }<br>
><br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>