Re: [PATCH, 2/4] Replace error_at with assert in build_va_arg

2016-09-04 Thread Tom de Vries
On 03/09/16 17:37, Andreas Schwab wrote: On Aug 24 2016, Tom de Vries wrote: > Replace error_at with assert in build_va_arg > > 2016-08-22 Tom de Vries > >* c-common.c (build_va_arg): Replace first argument type error >with assert.

Re: [PATCH, 2/4] Replace error_at with assert in build_va_arg

2016-09-03 Thread Andreas Schwab
http://gcc.gnu.org/ml/gcc-testresults/2016-09/msg00298.html Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something completely different."

Re: [PATCH, 2/4] Replace error_at with assert in build_va_arg

2016-09-03 Thread Tom de Vries
On 03/09/16 17:37, Andreas Schwab wrote: On Aug 24 2016, Tom de Vries wrote: > Replace error_at with assert in build_va_arg > > 2016-08-22 Tom de Vries > >* c-common.c (build_va_arg): Replace first argument type error >with assert.

Re: [PATCH, 2/4] Replace error_at with assert in build_va_arg

2016-09-03 Thread Andreas Schwab
On Aug 24 2016, Tom de Vries wrote: > Replace error_at with assert in build_va_arg > > 2016-08-22 Tom de Vries > > * c-common.c (build_va_arg): Replace first argument type error > with assert.

Re: [PATCH, 2/4] Replace error_at with assert in build_va_arg

2016-08-29 Thread Joseph Myers
On Wed, 24 Aug 2016, Tom de Vries wrote: > Hi, > > at the start of build_va_arg, we error out if there's an invalid va_list > argument. > > Therefore, the following checks on va_list type are not user errors, but > internal errors. > > This patch changes the user errors into internal errors. >

[PATCH, 2/4] Replace error_at with assert in build_va_arg

2016-08-24 Thread Tom de Vries
Hi, at the start of build_va_arg, we error out if there's an invalid va_list argument. Therefore, the following checks on va_list type are not user errors, but internal errors. This patch changes the user errors into internal errors. Bootstrapped and reg-tested on x86_64. OK for trunk?