<br><br><div class="gmail_quote">On Tue, Jul 8, 2008 at 1:16 AM, 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;">
Thanks :-)<br>
<br>
One last thing. It seems like the patch has a lot of whitespace<br>
changes. Could you create the diff with the -b option?</blockquote><div>Sure, I'll keep this in mind, thanks!<br><br>-Rajika <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<font color="#888888"><br>
-bw<br>
</font><div><div></div><div class="Wj3C7c"><br>
On Mon, Jul 7, 2008 at 11:49 AM, Rajika Kumarasiri <<a href="mailto:rajikacc@gmail.com">rajikacc@gmail.com</a>> wrote:<br>
> hi Bill,<br>
> Thanks for the comments. I'll update the patch according to that.<br>
><br>
> -Rajika<br>
><br>
> On Mon, Jul 7, 2008 at 11:56 PM, Bill Wendling <<a href="mailto:isanbard@gmail.com">isanbard@gmail.com</a>> wrote:<br>
>><br>
>> On Mon, Jul 7, 2008 at 11:10 AM, Rajika Kumarasiri <<a href="mailto:rajikacc@gmail.com">rajikacc@gmail.com</a>><br>
>> 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>
>> Hi Rajika,<br>
>><br>
>> A few stylistic comments. I'll let others comment on the algorithm for<br>
>> 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>
><br>
><br>
><br>
> --<br>
> comp.lang.c - <a href="http://groups.google.com/group/comp.lang.c/topics" target="_blank">http://groups.google.com/group/comp.lang.c/topics</a><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>
><br>
><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>
</div></div></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>