Hi Alexander,

just a couple of nits:

> gcc/testsuite/ChangeLog:
>
>         * gcc.target/i386/mpx/PR79988.c: New test.

We usually don't use capital PR in testcase names.  Please use pr79988.c
instead, matching the other files there.

Also, both ChangeLog entries should include a PR reference like so:

        PR middle-end/79988
        * gcc.target/i386/mpx/pr79988.c: New test.

so the commit messages are automatically forwarded to bugzilla.

> gcc/ChangeLog:
> 
>         * tree-chkp.c (chkp_gimple_call_builtin_p):
> Remove gimple_call_builtin_p call to avoid the call
> of gimple_builtin_call_types_compatible_p. this will
> strip the checks for address spaces, which can be skipped
> without loosing the functionality

ChangeLog entries only describe *what* changed, not *why*.

Thanks.
        Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University

Reply via email to