hi Bill,<br>Thanks for the comments. I'll update the patch according to that.<br><br>-Rajika<br><br><div class="gmail_quote">On Mon, Jul 7, 2008 at 11:56 PM, Bill Wendling <<a href="mailto:isanbard@gmail.com">isanbard@gmail.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="Ih2E3d">On Mon, Jul 7, 2008 at 11:10 AM, Rajika Kumarasiri <<a href="mailto:rajikacc@gmail.com">rajikacc@gmail.com</a>> wrote:<br>

> hello everybody,<br>
><br>
> I have added few improvements to my patch. Please review the new patch<br>
> directly.<br>
> Thanks!<br>
><br>
</div>Hi Rajika,<br>
<br>
A few stylistic comments. I'll let others comment on the algorithm for now:<br>
<br>
     if ($VERBOSE) {<br>
       print "( time -p $SVNCMD/llvm/trunk llvm; cd llvm/projects ; " .<br>
-            "$SVNCMD/test-suite/trunk llvm-test ) > $COLog 2>&1\n";<br>
+            "$SVNCMD/test-suite/trunk llvm-test ; cd ../../;" .<br>
+           "$SVNCMD/llvm-gcc-4.2/trunk dst-directory ) > $COLog 2>&1\n";<br>
     }<br>
     system "( time -p $SVNCMD/llvm/trunk llvm; cd llvm/projects ; " .<br>
-          "$SVNCMD/test-suite/trunk llvm-test ) > $COLog 2>&1\n";<br>
+          "$SVNCMD/test-suite/trunk llvm-test ; cd ../../;" .<br>
+         "$SVNCMD/llvm-gcc-4.2/trunk dst-directory ) > $COLog 2>&1\n";<br>
<br>
This could be done with something like this:<br>
<br>
     my $cmd = "( time -p $SVNCMD/llvm/trunk llvm; cd llvm/projects ; " .<br>
              "$SVNCMD/llvm-gcc-4.2/trunk dst-directory; " .<br>
              "cd llvm/projects; " .<br>
              "$SVNCMD/test-suite/trunk llvm-test ) > $COLog 2>&1\n";<br>
<br>
     print $cmd if ($VERBOSE);<br>
     system $cmd;<br>
<br>
This could be done in other spots that do similar things as well.<br>
Also, why not name the LLVM-GCC source directory something other than<br>
"dst-directory"? Maybe "llvm-gcc.src" or similar? You add a few<br>
extraneous newlines in the code. Please remove those.<br>
<br>
-    my $CVSCMD = "$NICE cvs $CVSOPT -d $CVSRootDir co -P $CVSCOOPT";<br>
+    my $CVSCMD = "$NICE cvs $CVSOPT -d $CVSRootDir co -P $CVSCOOPT";<br>
# TODO, do we still maintain the CVS tree ?<br>
<br>
80-column violation here and in other places. Put the comment before<br>
here. (And, no, we don't maintain the CVS tree anymore.)<br>
<br>
-bw<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</blockquote></div><br><br clear="all"><br>-- <br>comp.lang.c - <a href="http://groups.google.com/group/comp.lang.c/topics">http://groups.google.com/group/comp.lang.c/topics</a>