[PATCH] PR other/44286, with mips, sparc, C++, Fortran, gcov, frontend bits
NightStrike
nightstrike@gmail.com
Mon Jun 7 16:19:00 GMT 2010
On Mon, May 31, 2010 at 2:42 PM, Richard Sandiford
<rdsandiford@googlemail.com> wrote:
> Ralf Wildenhues <Ralf.Wildenhues@gmx.de> writes:
>> I glanced at code places matching ', *\.\.\. *)'.
>>
>> Bootstrapped x86_64-unknown-linux-gnu, regtest is running. I still need
>> to test the mips and sparc changes.
>>
>> Only question I have is whether an ATTRIBUTE_SENTINEL_N(N) is preferable
>> to ATTRIBUTE_SENTINEL_1. sentinel and sentinel(N) were introduced in
>> the same GCC version AFAICS.
>>
>> There are a few more functions which use an integer zero sentinel, but
>> only where all vararg arguments are integers, so it doesn't make sense
>> to change that just so the attribute applies.
>>
>> OK to commit if no regressions show up?
>
> The MIPS parts are OK, thanks.
>
> Richard
>
Ping for the rest of this patch.
Bug still open: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44286
More information about the Gcc-patches
mailing list