that
explain the semantics of the arguments (by name) and any return value.
+/* If current scope is a label scope, pop it from block info stack
+ and close it's compound statement. */
its not it's.
--
Joseph S. Myers
jos...@codesourcery.com
I don't have any comments on this patch.
--
Joseph S. Myers
jos...@codesourcery.com
believe linking with -lm is the default in the testsuite so shouldn't
need specifying explicitly.
--
Joseph S. Myers
jos...@codesourcery.com
to intN support.
--
Joseph S. Myers
jos...@codesourcery.com
* c-typeck.c (parser_build_binary_op): Don't call
warn_logical_not_parentheses if the RHS is TRUTH_NOT_EXPR.
* c-c++-common/pr49706-2.c: New test.
OK.
--
Joseph S. Myers
jos...@codesourcery.com
Andrew MacLeod amacl...@redhat.com
PR c/61553
* c-common.c (get_atomic_generic_size): Don't segfault if the
type doesn't have a size.
* c-c++-common/pr61553.c: New test.
OK.
--
Joseph S. Myers
jos...@codesourcery.com
builds if it is in fact in use.
--
Joseph S. Myers
jos...@codesourcery.com
).
--
Joseph S. Myers
jos...@codesourcery.com
type qualifiers in
__typeof__ for atomic types.
gcc/testsuite/ChangeLog
2014-06-26 Sebastian Huber sebastian.hu...@embedded-brains.de
* gcc.dg/typeof-2.c: New testcase.
OK.
--
Joseph S. Myers
jos...@codesourcery.com
in the callers to ensure this.)
--
Joseph S. Myers
jos...@codesourcery.com
, const_rtx expr, void *cuip)
{
cselib_val *oval = cselib_lookup (oloc, GET_MODE (oloc), 0, VOIDmode);
- gcc_assert (oval != v);
+ if (oval == v)
+ return;
gcc_assert (REG_P (oloc) || MEM_P (oloc));
if (oval !cselib_preserved_value_p (oval))
--
Joseph S
years back I fixed exactly this kind of bug in
elfutils - so -Wsizeof-array-argument might be indeed useful. (The warning
didn't
trigger during GCC bootstrap though.)
Jason/Joseph, could you please look at the C++, resp. C FE parts?
The C changes are OK.
--
Joseph S. Myers
jos
allocating registers (which might not always
correspond neatly to multiples of BITS_PER_UNIT), TYPE_MODE is what's
relevant; otherwise, you're allocating whole bytes in memory and can use
TYPE_SIZE_UNIT.
--
Joseph S. Myers
jos...@codesourcery.com
TYPE_SIZE.
--
Joseph S. Myers
jos...@codesourcery.com
this.
--
Joseph S. Myers
jos...@codesourcery.com
.
--
Joseph S. Myers
jos...@codesourcery.com
with the documentation amended to make clear this is for the cases not
covered by -Wno-pointer-sign (which in ISO C terms are just as
incompatible as the cases covered by the new option).
--
Joseph S. Myers
jos...@codesourcery.com
and
-Wno-pointer-to-int-cast.
--
Joseph S. Myers
jos...@codesourcery.com
that it is always the case that ordinary operations such as
commits, branching, tagging etc. do serve to commit, branch, tag etc. the
entire source tree in the GCC repository, when done by anyone with write
access to it and without needing write access to any external repository.
--
Joseph S
to make_relative_prefix.
The relevant information for relocation will also need passing to cpplib
so that its call to bindtextdomain also gets a relocated path.
--
Joseph S. Myers
jos...@codesourcery.com
On Wed, 9 Jul 2014, Mingjie Xing wrote:
2014-07-09 Mingjie Xing mingjie.x...@gmail.com
* texi2pod.pl (postprocess): Move command process for '@sc' to the
front of '@dfn'. Add a new command process for '@t{...}', just print
the content.
OK.
--
Joseph S. Myers
, with libgcc's soft-fp copy then
being updated from glibc.
--
Joseph S. Myers
jos...@codesourcery.com
.html would be needed. I
also don't understand what the sign bit manipulation is for; the low part
(before renormalization) should be the result of the subtraction (whose
sign may or may not be that of the high part), without any such
manipulation.
--
Joseph S. Myers
jos...@codesourcery.com
* targets
more like other architectures in this regard - each OS's headers define
the specs for that OS without the added indirection.
(I'm doubtful of the current use of -mads -myellowknife -mmvme as well -
other architectures avoid having such special compiler options for each
BSP.)
--
Joseph S
).
--
Joseph S. Myers
jos...@codesourcery.com
of such a migration to decide
whether they should stay or be removed. (There aren't that many of them
to review.)
--
Joseph S. Myers
jos...@codesourcery.com
* gcc.dg/Wdesignated-init.c: New file.
OK.
--
Joseph S. Myers
jos...@codesourcery.com
target dependencies
in bits built once.)
--
Joseph S. Myers
jos...@codesourcery.com
in install.texi. I see no such documentation
in this patch.
--
Joseph S. Myers
jos...@codesourcery.com
On Tue, 30 Sep 2014, Richard Earnshaw wrote:
GCC is written in C++ these days, so technically, you need the C++
standard :-)
And, while C++14 requires plain int bit-fields to be signed, GCC is
written in C++98/C++03.
--
Joseph S. Myers
jos...@codesourcery.com
extracted by exgettext for translation.
--
Joseph S. Myers
jos...@codesourcery.com
)
+{
+ int i;
+ for (i = 0; i argc; i++)
+ printf (argc[%i]: %s\n, i, argv[i]);
+}
--
Joseph S. Myers
jos...@codesourcery.com
by !flag_gnu89_inline.
Bootstrapped/regtested on x86_64-linux, ok for trunk?
OK.
--
Joseph S. Myers
jos...@codesourcery.com
if you link in GCC's other libraries statically, it would seem a good idea
for the documentation of this option to make that explicit.
--
Joseph S. Myers
jos...@codesourcery.com
* c-convert.c (convert): Use error_operand_p.
* c-typeck.c (require_complete_type): Likewise.
(really_atomic_lvalue): Likewise.
(digest_init): Likewise.
(handle_omp_array_sections_1): Likewise.
OK.
--
Joseph S. Myers
jos...@codesourcery.com
On Mon, 6 Oct 2014, Steve Kargl wrote:
On Mon, Oct 06, 2014 at 08:38:14PM +, Joseph S. Myers wrote:
Since -static-libquadmath introduces LGPL requirements on redistributing
the resulting binaries (that you provide source or relinkable object files
to allow relinking with modified
If adding a new option, you need to document it in invoke.texi.
--
Joseph S. Myers
jos...@codesourcery.com
Patches adding new -march= values need to update invoke.texi.
--
Joseph S. Myers
jos...@codesourcery.com
On Tue, 7 Oct 2014, Marek Polacek wrote:
2014-10-07 Marek Polacek pola...@redhat.com
PR c/59717
* c-decl.c (header_for_builtin_fn): New function.
(implicitly_declare): Suggest which header to include.
* gcc.dg/pr59717.c: New test.
OK.
--
Joseph S. Myers
jos
.
I'd like to hear from Joseph ;).
I approve of the change in default (I just don't think it's a change to
make on my say-so alone).
--
Joseph S. Myers
jos...@codesourcery.com
(min_align_of_type): Don't decrease alignment
through BIGGEST_FIELD_ALIGNMENT or ADJUST_FIELD_ALIGN if
TYPE_USER_ALIGN is set.
* gcc.target/i386/pr63495.c: New test.
OK.
--
Joseph S. Myers
jos...@codesourcery.com
On Wed, 15 Oct 2014, David Malcolm wrote:
As for the bindir in site.exp, Joseph asked me when the library
invokes a driver to convert from .s to .so to:
On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote:
* use the $(target_noncanonical)-gcc-$(version) name for the
driver rather
of
the type (either in the type itself, or on the side) that *only* means an
error has been given for the type being incomplete, rather than inserting
error_mark_node into the type.
--
Joseph S. Myers
jos...@codesourcery.com
On Wed, 15 Oct 2014, Jeff Law wrote:
On 10/15/14 15:46, Joseph S. Myers wrote:
On Wed, 15 Oct 2014, Marek Polacek wrote:
We've got a complaint that the dereferencing pointer to incomplete
type error is printed for all occurrences of the incomplete type,
which is too verbose. Also
logically correct for
bitsizetype). The other changes are OK.
--
Joseph S. Myers
jos...@codesourcery.com
the existing logic, bitsizetype is expected to be
bigger than sizetype, even if inefficient.)
--
Joseph S. Myers
jos...@codesourcery.com
Changes to architecture-independent files must use
architecture-independent conditionals, so __BYTE_ORDER__ not __ARMEB__.
--
Joseph S. Myers
jos...@codesourcery.com
literal, we know
the code is invalid in ISO C terms, so it's safe to diagnose it then
interpret it in a sensible way.)
--
Joseph S. Myers
jos...@codesourcery.com
warn_implicit_int in such
a way that -Wimplicit and -Wimplicit-int should work as intended
(following the rule that more specific option takes precedence
over the less specific). There should be no changes in GNU89
mode.
Bootstrapped/regtested on x86_64-linux, ok for trunk?
OK.
--
Joseph S
On Fri, 17 Oct 2014, Manuel López-Ibáñez wrote:
Thus, I drafted some guidelines
at:https://gcc.gnu.org/wiki/Better_Diagnostics#guidelines
Please, could you take a look and comment whether I got it right/wrong?
Yes, that looks right to me.
--
Joseph S. Myers
jos...@codesourcery.com
relevance to systems people are building GCC on any more.
I don't have any comments on the other parts of this patch.
--
Joseph S. Myers
jos...@codesourcery.com
as it never gets
executed).
--
Joseph S. Myers
jos...@codesourcery.com
in C99 and add pedwarn
for it.
* gcc.dg/pr61096-1.c: Change dg-error into dg-warning.
* gcc.dg/pr63567-1.c: New test.
* gcc.dg/pr63567-2.c: New test.
OK.
--
Joseph S. Myers
jos...@codesourcery.com
Although Sphinx isn't a build dependency, as a dependency for
regenerating checked-in files I think it should be documented in
install.texi (like autoconf, gettext, etc.).
--
Joseph S. Myers
jos...@codesourcery.com
on fclose (or fwrite)).
--
Joseph S. Myers
jos...@codesourcery.com
could
depend on STRIP_[SIGN_]NOPS calls.
--
Joseph S. Myers
jos...@codesourcery.com
translation).
--
Joseph S. Myers
jos...@codesourcery.com
give an error *if* there is compile-time
undefined behavior (not if the not-definedness is something other than
undefined behavior, or is undefined behavior only if the code in question
is executed, although it will still give a warning for such cases if
-pedantic does).
--
Joseph S. Myers
jos
.
--
Joseph S. Myers
jos...@codesourcery.com
things should be structured to avoid
splitting up English fragments like that.
--
Joseph S. Myers
jos...@codesourcery.com
for it.
* gcc.dg/pr63567-3.c: New test.
* gcc.dg/pr63567-4.c: New test.
OK.
--
Joseph S. Myers
jos...@codesourcery.com
system headers directly if
they are included by system.h) even if no other GCC headers are useful in
any way.
--
Joseph S. Myers
jos...@codesourcery.com
On Mon, 20 Oct 2014, David Malcolm wrote:
On Fri, 2014-10-17 at 21:52 +, Joseph S. Myers wrote:
[...snip static linkage discussion...]
The dump file handling appears to have no I/O error checking (no checking
for error on fopen, nothing obvious to prevent fwrite to a NULL m_file
On Mon, 20 Oct 2014, David Malcolm wrote:
+Necessary to regenerate @file{jit/docs/_build/texinfo} from the .rst
I'd say @file{.rst}, but otherwise looks OK to me.
--
Joseph S. Myers
jos...@codesourcery.com
in
TARGET_ATOMIC_ASSIGN_EXPAND_FENV that will need that (along with libgcc
changes to make libgcc's copies of the soft-fp functions into compat
symbols when they are available in glibc). That's nothing to do with the
timeout issue.
--
Joseph S. Myers
jos...@codesourcery.com
On Tue, 25 Mar 2014, Marek Polacek wrote:
We should mention this GNU C extension in the manual.
Ok?
2014-03-25 Marek Polacek pola...@redhat.com
PR c/37428
* doc/extend.texi (C Extensions): Mention variable-length arrays in
a structure/union.
OK.
--
Joseph S
): Describe what happens to omitted
field members.
OK.
--
Joseph S. Myers
jos...@codesourcery.com
@printindex cp
--
Joseph S. Myers
jos...@codesourcery.com
like that;
either the code should get a compile-time error (that's what I suggest),
or it should generate output that links but has undefined behavior at
runtime.
--
Joseph S. Myers
jos...@codesourcery.com
-mfpmath=sse -mfma -mtune=generic } */
+/* { dg-options -O3 -Wno-attributes -mfpmath=sse -mfma -mtune=generic
-mno-fma4 } */
/* Test that the compiler properly optimizes floating point multiply
and add instructions into FMA3 instructions. */
--
Joseph S. Myers
jos...@codesourcery.com
objections within 24 hours.
--
Joseph S. Myers
jos...@codesourcery.com
(working copy)
@@ -1,5 +1,5 @@
/* { dg-do compile } */
-/* { dg-options -O3 -dp -mavx -mavx256-split-unaligned-store -fno-common } */
+/* { dg-options -O3 -dp -mavx -mavx256-split-unaligned-store
-mno-prefer-avx128 -fno-common } */
#define N 1024
--
Joseph S. Myers
jos...@codesourcery.com
the appropriate effective-target there, depending on
whether the condition is one on the ABI or which register size is being
used, which affects how x32 should be counted).
--
Joseph S. Myers
jos...@codesourcery.com
-times movdqa 8 { target { ! { ia32 } } } } }
*/
/* { dg-final { scan-assembler-times movdqa 4 { target { ia32 } } } } */
--
Joseph S. Myers
jos...@codesourcery.com
avx_test (char **cp, char **ep)
--
Joseph S. Myers
jos...@codesourcery.com
has behavior fully defined
to exit with status 0).
--
Joseph S. Myers
jos...@codesourcery.com
.
int
main ()
{
1 || ({ A: 0; });
void *L[] = { A };
}
--
Joseph S. Myers
jos...@codesourcery.com
.
[*] At least right now. There's a PR about -Wmissing-field-initializers=2
that should warn even for designated initializers.
Regtested/bootstrapped on x86_64-linux, ok for trunk?
OK.
--
Joseph S. Myers
jos...@codesourcery.com
and printing them in hex, which is
easy for values of any precision.
--
Joseph S. Myers
jos...@codesourcery.com
should I think also cover incomplete union types, and pointer
subtraction for all the different cases of incomplete types, and the +=
and -= operators (i.e. cover invalid arithmetic on pointers to incomplete
types more thoroughly, even if not actually affected by this patch).
--
Joseph S. Myers
defining things in the
implementation namespace, with the public sys/types.h being what then
includes sys/_types.h and does typedef __foo_t foo_t; or similar to
provide the public POSIX types that aren't in ISO C.)
--
Joseph S. Myers
jos...@codesourcery.com
to be removed (and corresponding
more detailed error expectations added), or a separate test needs adding
for this erroneous use of this attribute (that separate test not using
dg-prune-output).
--
Joseph S. Myers
jos...@codesourcery.com
seen
main (int, ...). But it seemed wrong to not to warn on that.
Thanks,
As a matter of QoI we should also diagnose use of _Atomic in the return
type or argument types of main (something I deferred doing in the initial
_Atomic support).
--
Joseph S. Myers
jos...@codesourcery.com
if pedantic, please explain further why this is the
case.
--
Joseph S. Myers
jos...@codesourcery.com
have changed my mind; goto seems clearer here (to me).
Indeed. Please revert.
--
Joseph S. Myers
jos...@codesourcery.com
zh_CN.po from the TP (whether resulting
from changes by translators, or an automatic merge with a new .pot file),
that will be imported verbatim (to both trunk and 4.9 branch), whether or
not it includes this change.
--
Joseph S. Myers
jos...@codesourcery.com
need a
prototypes for them.
Bootstrapped on x86_64-linux, ok for trunk?
OK.
--
Joseph S. Myers
jos...@codesourcery.com
type may help you
find where the missing initializer is).
--
Joseph S. Myers
jos...@codesourcery.com
suggest names such as ARM_FE_* that won't conflict with the
host's system headers.
--
Joseph S. Myers
jos...@codesourcery.com
this case, decide what the best
semantics are and ensure the testsuite verifies that they are implemented.
--
Joseph S. Myers
jos...@codesourcery.com
, but
Annex F says they raise invalid and return an unspecified value.)
--
Joseph S. Myers
jos...@codesourcery.com
that.)
--
Joseph S. Myers
jos...@codesourcery.com
).
--
Joseph S. Myers
jos...@codesourcery.com
On Thu, 1 May 2014, DJ Delorie wrote:
Do we have a routine to do that already?
I don't know.
--
Joseph S. Myers
jos...@codesourcery.com
for stddef.h inclusion, as long as the machine/_types.h
is OK.
http://grok.dragonflybsd.org/xref/dragonfly/sys/sys/types.h
That's definitely not correct to include in stddef.h; it defines lots of
types outside the ISO C namespace.
--
Joseph S. Myers
jos...@codesourcery.com
On Thu, 1 May 2014, Jeff Law wrote:
On 05/01/14 17:59, Joseph S. Myers wrote:
On Thu, 24 Apr 2014, Tobias Burnus wrote:
Jeff, are you sure that this will not be overridden? I think the
translations
are always synced from http://translationproject.org/team/zh_CN.html ?
Yes
On Fri, 2 May 2014, Marek Polacek wrote:
On Thu, May 01, 2014 at 10:54:18PM +, Joseph S. Myers wrote:
I think the comment on this function should be updated to explain the
interface contract when an incomplete (or function) type is passed (i.e.
return size_one_node, caller
warning_at calls.
OK.
--
Joseph S. Myers
jos...@codesourcery.com
On Fri, 2 May 2014, Marek Polacek wrote:
Wsizeof-pointer-memaccess option is missing a description in
the c.opt file. This patch adds something in there.
Ok?
OK.
--
Joseph S. Myers
jos...@codesourcery.com
feature).
So try removing that #include. If that causes problems, investigate based
on the actual problems seen.
--
Joseph S. Myers
jos...@codesourcery.com
1 - 100 of 1549 matches
Mail list logo
