I don't understand some of gcc-4.1-20070514, a patch here.

[J.C. Pizarro]

2007/5/19, i wrote at http://gcc.gnu.org/ml/gcc/2007-05/msg00470.html:
Hi developers,

for this http://gcc.gnu.org/ml/gcc/2007-05/msg00451.html

you have this nice cleanup's patch of gcc/loop.c that transliterates the logic
 of the uses of the loop_invariant_p (..) and consec_sets_invariant_p (..)
 functions.

I've patched it, builded and executed, and again again with this patched gcc.
It's OK.

It will be more readable and comprehensible. I did patch the comprehension!

To the expert person only, please, read now the loop algorithms that
uses *_OF_LOOP_P if there is any failed logic or not.

Sincerely yours, J.C.


2007/5/19, Eric Botcazou <ebotcazou@libertysurf.fr> wrote:
> > you have this nice cleanup's patch of gcc/loop.c that transliterates the
> > logic of the uses of the loop_invariant_p (..) and consec_sets_invariant_p
> > (..) functions.
>
> First of all, patches should be posted to gcc-patches@, not to this list.
>
> > I've patched it, builded and executed, and again again with this patched
> > gcc. It's OK.
>
> This is not sufficient.  Patch submission must comply with strict rules that
> are listed at http://gcc.gnu.org/contribute.html.
>
> > To the expert person only, please, read now the loop algorithms that
> > uses *_OF_LOOP_P if there is any failed logic or not.
>
> loop.c is gone in the mainline sources.  Patching it on the 4.1 branch is
> allowed only if you have a testcase that exposes a serious bug.
>
> --
> Eric Botcazou

I've read http://gcc.gnu.org/contribute.html and agreed it.

I'd modified the patch: '//' to '/*' .. '*/' to not abort the build.

I've made the following

make bootstrap
make -k check

through my script gcc_build.sh and it's OK. Too, i did

grep -v '^PASS:' gcc.sum > gcc_grep_no_PASS.sum

Sincerely yours, J.C. ;)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gcc_build.sh
Type: application/x-sh
Size: 577 bytes
Desc: not available
URL: <https://gcc.gnu.org/pipermail/gcc/attachments/20070519/b501c743/attachment.sh>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gcc_build.log.7z
Type: application/octet-stream
Size: 31227 bytes
Desc: not available
URL: <https://gcc.gnu.org/pipermail/gcc/attachments/20070519/b501c743/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gcc-4.1-20070514_renaming_logic_of_loop_invariant_p.patch
Type: application/octet-stream
Size: 13119 bytes
Desc: not available
URL: <https://gcc.gnu.org/pipermail/gcc/attachments/20070519/b501c743/attachment-0001.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gcc.sum.7z
Type: application/octet-stream
Size: 91792 bytes
Desc: not available
URL: <https://gcc.gnu.org/pipermail/gcc/attachments/20070519/b501c743/attachment-0002.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gcc_grep_no_PASS.sum.gz
Type: application/x-gzip
Size: 3115 bytes
Desc: not available
URL: <https://gcc.gnu.org/pipermail/gcc/attachments/20070519/b501c743/attachment.bin>