;
+class rtl_opt_pass;
+
+namespace gcc {
+ class context;
+}
+
This break Ada.
../../xgcc -B../../ -c -DIN_GCC -DUSE_LIBUNWIND_EXCEPTIONS -O2 -g -W -Wall \
-iquote /usr/local/gcc/gcc-20130807/gcc \
-iquote . -iquote .. -iquote ../.. -iquote
/usr/local/gcc
Hello,
in the discussion about the PR middle-end/57748 it became obvious that the ARM
target architecture should define a value for MALLOC_ABI_ALIGNMENT, because
otherwise the default is simply word aligned, which causes sub-optimal code, at
least for the neon fpu.
This simple patch fixes PR
the option is designed for purpose like this.
That's great, thanks!
Michael
David
On 6 August 2013 20:42, Xinliang David Li davi...@google.com wrote:
Corrected two small problems reported by the style checker (The
warnings about the EnumValue for options in stringopt.opt are not
valid).
Hi Jason,
On Mon, 05 Aug 2013 17:26:12 -0400, Jason Merrill wrote:
On 08/04/2013 07:45 PM, Adam Butcher wrote:
What should I do about the symtab nullptr issue?
(http://gcc.gnu.org/ml/gcc-patches/2013-07/msg00043.html) Should I
leave the workaround in my patch set as a standalone commit to
Hi,
On 08/07/2013 01:08 AM, Jason Merrill wrote:
On 08/06/2013 06:14 PM, Paolo Carlini wrote:
Ah I see, thanks. We reject this before and after the patch. Shall I add
this variant to the new testcase?
Sure.
Thanks. Thus after a further round of testing I'm going to apply the below.
Thanks,
On Tue, Aug 06, 2013 at 07:07:27PM -0400, Jason Merrill wrote:
I think, what we could do, is to tweak verify_constant like this:
+ /* This is to handle e.g. the goofy 'case 0 * (1 / 0)' case. */
+ if (flag_sanitize SANITIZE_UNDEFINED
+ TREE_CODE (t) == CALL_EXPR
+
On Tue, Aug 6, 2013 at 6:42 PM, Ian Lance Taylor i...@google.com wrote:
I committed this patch that I think should fix this problem.
Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committed to mainline and 4.8 branch.
2013-08-06 Ian Lance Taylor i...@google.com
*
trunk was merged into c++-concepts branch at revision 201560.
I resolved some conflicts in cp caused by recent merges and ongoing work
on trunk.
-- Gaby
Richard Sandiford rdsandif...@googlemail.com wrote:
PR 58079 is about the do_SUBST assert:
/* Sanity check that we're replacing oldval with a CONST_INT
that is a valid sign-extension for the original mode. */
gcc_assert (INTVAL (newval)
== trunc_int_for_mode
Hi,
I'm applying this mainline and 4_8-branch. Tested x86_64-linux.
Thanks,
Paolo.
//
2013-08-07 Paolo Carlini paolo.carl...@oracle.com
PR libstdc++/58098
* include/bits/random.h (cauchy_distribution::min,
extreme_value_distribution::min,
Revision 201555 breaks boostrap on x86_64-apple-darwin10:
...
Checking multilib configuration for libvtv...
make all-recursive
Making all in testsuite
/bin/sh: line 0: cd: testsuite: No such file or directory
make[4]: *** [all-recursive] Error 1
make[3]: *** [all] Error 2
make[2]: ***
The mips*-*-* targets are not building. It looks like the mips reorg
pass (pass_mips_machine_reorg2) in config/mips/mips.c was not converted
and/or was not converted correctly.
Likewise for the SPARC targets, because of the same issue. David, could you
take a quick look? Thanks in advance.
Hi all,
In PR58088 the constant folder goes into an infinite recursion and runs out of
stack space because of two conflicting optimisations:
(X * C1) C2 plays dirty when nested inside an IOR expression like so: ((X *
C1) C2) | C4. One can undo the other leading to an infinite recursion.
Thanks
On Tue, 2013-08-06 at 22:01 -0500, Peter Bergner wrote:
I'll bootstrap and regtest the change
below and commit it as an obvious fix when that is done.
Everything looked as as expected. Committed.
Peter
I think it's r201508, but in any case, I'm attaching a patch which fixes
this build error. Only very lightly tested so far, with configure
--target=sparc-linux with buildhost x86_64. Was able to build a cc1
and step through the changed code in the debugger, though am getting
cc1: error: no
Hi
This has been around for a while as PR52466 which has
a patch. I have attached the patch. Is it OK to apply to
the open branches?
2013-08-07 Joel Sherrill joel.sherr...@oarcorp.com
* configure.ac (lm32*-*-*): Use SJLJ exceptions.
--
Joel Sherrill, Ph.D. Director of
On 07/08/13 01:25, Joseph S. Myers wrote:
On Tue, 30 Jul 2013, Richard Earnshaw wrote:
Most arm target configs now require a 64-bit HW-int. Unfortunately a few of
the older, less commonly used config targets do not. The code in arm.c now
pretty much requires that a 64-bit HW-int is used,
Hi all,
here is a small regression-fix patch for a problem with procedure
pointers and the PURE attribute, for details see the PR. In essence:
gfc_compare_interfaces is asymmetric in the two interfaces it compares
(e.g. regarding the PURE attribute), and we must not symmetrize it by
calling it
On 08/07/2013 06:06 AM, Marek Polacek wrote:
I might've misunderstood what you mean. If we drop the hunk above,
then we'll call
error (%q+E is not a constant expression, t);
so, we'll print is not a constant expression no matter what
Yes, that's fine; 1/0 is not a constant expression,
This issue is present in 4.8.1 as well as trunk. However, I think 4.8
uses a different API for double_int, so this patch will need to be reworked
for
4.8.
Actually, I was confused. This patch applies to 4.8.1 as well and fixes the
issue. Passes bootstrap on x86_64-linux-gnu and testing
On Tue, Aug 06, 2013 at 05:24:08PM -0400, Jason Merrill wrote:
On 08/05/2013 07:24 AM, Marek Polacek wrote:
On Sat, Aug 03, 2013 at 12:24:32PM -0400, Jason Merrill wrote:
Where are the SAVE_EXPRs coming from? It doesn't seem to me that x
needs to be wrapped in a SAVE_EXPR at all in this case.
Ping?
David
On Sat, Aug 3, 2013 at 12:36 PM, Xinliang David Li davi...@google.com wrote:
Hi, GCC/i386 currently has about 73 boolean parameters/knobs (defined
in ix86_tune_features[], indexed by ix86_tune_indices) to perform
micro-arch specific performance tuning. However such settings are
Ping?
David
On Sat, Aug 3, 2013 at 12:36 PM, Xinliang David Li davi...@google.com wrote:
Hi, GCC/i386 currently has about 73 boolean parameters/knobs (defined
in ix86_tune_features[], indexed by ix86_tune_indices) to perform
micro-arch specific performance tuning. However such
On Wed, 2013-08-07 at 15:25 +0200, Eric Botcazou wrote:
Looks good, please install if not already done.
Thanks; I've now committed this to trunk as r201569.
In poking at another set of fails encountered with an i686-elf toolchain
(http://gcc.gnu.org/ml/gcc/2013-08/msg00081.html), I find that complex double
returns and PIC do not play together.
Libgcc's __muldc3 was attempting to return the value in registers, whereas every
caller expected it to
On 13-08-06 3:11 PM, Richard Henderson wrote:
On 08/06/2013 03:57 AM, Kirill Yukhin wrote:
On 05 Aug 09:55, Richard Henderson wrote:
On 08/05/2013 08:07 AM, Kirill Yukhin wrote:
Hello Richard, Vlad,
On 31 Jul 06:26, Richard Henderson wrote:
On 07/31/2013 05:02 AM, Kirill Yukhin wrote:
On 08/07/2013 03:52 AM, Adam Butcher wrote:
But a cleaner way might be to extend the processing
template declaration state from lambda declarator all the way to the end of the
lambda body. This would match with the scenario that occurs with a standard
in-class member function template
On Wed, Aug 07, 2013 at 10:24:59AM -0400, Jason Merrill wrote:
On 08/07/2013 06:06 AM, Marek Polacek wrote:
I might've misunderstood what you mean. If we drop the hunk above,
then we'll call
error (%q+E is not a constant expression, t);
so, we'll print is not a constant expression no
On Wed, Aug 7, 2013 at 8:37 AM, Jan Hubicka hubi...@ucw.cz wrote:
Ping?
David
On Sat, Aug 3, 2013 at 12:36 PM, Xinliang David Li davi...@google.com
wrote:
Hi, GCC/i386 currently has about 73 boolean parameters/knobs (defined
in ix86_tune_features[], indexed by ix86_tune_indices) to
Ping.
On Wed, Jul 31, 2013 at 4:34 PM, Easwaran Raman era...@google.com wrote:
I have a new patch that supersedes this. The new patch also fixes PR
tree-optimization/57393 and PR tree-optimization/58011. Bootstraps and
no test regression on x86_64/linux. Ok for trunk?
2013-07-31 Easwaran
libvtv was supposed to be automatically disabled for darwin; we are in
the process of trying to figure out why this did not work as it was
supposed to.
In the meantime, if you add --disable-libvtv explicitly to your
configure command, that should turn off the attempts to
configure/build it.
I'll
Fixed the do while formatting. Ok for trunk with this version?
thanks,
David
On Tue, Aug 6, 2013 at 2:42 AM, Jan Hubicka hubi...@ucw.cz wrote:
2013-08-02 Xinliang David Li davi...@google.com
* config/i386/stringop.def: New file.
* config/i386/stringop.opt: New file.
The usual x86 ports avoid this, because their must_return_in_mem hooks DTRT
already and we never fall into the default case.
So why not do the same for i686-elf?
I am very surprised this hasn't bitten someone before -- I presume this
never worked with i686-elf. It is possible this'll change
As requested, here is the patch to announce the vtable verification
feature on the main gcc.gnu.org web page. Is this ok to commit?
-- Caroline Tice
cmt...@google.com
wwwdocs.patch
Description: Binary data
hi Caroline,
A (very) quick look suggests that only *) results in a return of UNSUPPORTED
from libvtv/configure.tgt
Do you know if anyone is working on a Darwin port for this lib?
thanks,
Iain
On 7 Aug 2013, at 17:57, Caroline Tice wrote:
libvtv was supposed to be automatically disabled
On Wed, 2013-08-07 at 19:15 +0200, Basile Starynkevitch wrote:
Hello All,
It should be perfectly possible for a plugin argument to contain the
equal sign.
For plugin with scripting or DSL behavior (like MELT, see
http://gcc-melt.org/ for more) it is useful to pass some commands
or
Hello All,
It should be perfectly possible for a plugin argument to contain the
equal sign.
For plugin with scripting or DSL behavior (like MELT, see
http://gcc-melt.org/ for more) it is useful to pass some commands
or expressions to the plugin. For MELT we might want to run
gcc
Having poked further, I find this in the testsuite:
pr11001-strlen-2.c
--begin
register int regvar asm(%eax);
char *
do_copy (char *str)
{
return malloc (strlen (str) + 1);
}
--end
Is that even meaningful? The doc's for a global reg var say:
'Choose a register that is normally saved and
Hi Iain,
Thanks for the pointer (I had noticed this but I appreciate the help!).
I do not know of anyone working on a Darwin port for this at this time.
-- Caroline Tice
cmt...@google.com
On Wed, Aug 7, 2013 at 10:31 AM, Iain Sandoe i...@codesourcery.com wrote:
hi Caroline,
A (very) quick
ok for google branch.
David
On Tue, Aug 6, 2013 at 3:41 PM, Dehao Chen de...@google.com wrote:
Patch updated.
http://codereview.appspot.com/12079043
Thanks,
Dehao
On Fri, Aug 2, 2013 at 11:21 AM, Xinliang David Li davi...@google.com wrote:
More to follow.
David
static void
On Thu, Sep 13, 2012 at 7:46 PM, H.J. Lu hongjiu...@intel.com wrote:
Hi,
There is no reason why --eh-frame-hdr can't be used with static
executable on Linux. This patch enables --eh-frame-hdr for static
executable on Linux and adds an exception test for static executable.
Other platforms
A not-yet-reviewed libtool patch is needed for powerpc64le
shared-library support:
http://lists.gnu.org/archive/html/libtool-patches/2013-06/msg1.html
(See also
http://lists.gnu.org/archive/html/libtool-patches/2013-07/msg1.html)
This patch merges that change into the google/gcc-4_8
On Wed, Aug 7, 2013 at 11:31 AM, Paul Pluzhnikov ppluzhni...@google.com wrote:
On Wed, Aug 7, 2013 at 11:09 AM, Jan Kratochvil
jan.kratoch...@redhat.com wrote:
there should be a new case in:
libiberty/testsuite/demangle-expected
Thanks. I updated the patch.
--
Paul Pluzhnikov
I
On Wed, Aug 7, 2013 at 10:34 AM, Nathan Sidwell nat...@acm.org wrote:
Having poked further, I find this in the testsuite:
pr11001-strlen-2.c
--begin
register int regvar asm(%eax);
char *
do_copy (char *str)
{
return malloc (strlen (str) + 1);
}
--end
Is that even meaningful? The
Greetings,
I've been redirected here from binutils:
http://sourceware.org/ml/binutils/2013-08/msg00052.html
http://sourceware.org/ml/binutils/2013-08/msg00056.html
The following source:
templatetypename T static void f();
void g() { fint(); }
results in _Z1fIiEvv under g++, but in
On Wed, Aug 7, 2013 at 11:34 AM, Andrew Pinski pins...@gmail.com wrote:
I think this should also be send to the GCC List as libiberty is
officially maintained by GCC.
http://gcc.gnu.org/ml/gcc-patches/2013-08/msg00394.html
Thanks,
--
Paul Pluzhnikov
The Go frontend gave an incorrect missing return error message for a
type switch with a case with multiple types, as in case T1, T2:. This
patch fixes that problem. This patch also changes the missing return
error message to use the same text as the gc compiler, and reports it at
the end of the
On Tue, Aug 06, 2013 at 11:45:40PM +0200, Oleg Endo wrote:
On Mon, 2013-08-05 at 13:25 -1000, Richard Henderson wrote:
On 08/05/2013 12:32 PM, Oleg Endo wrote:
Thanks, committed as rev 201513.
4.8 also has the same problem. The patch applies on 4.8 branch without
problems and make
Hi,
the issue here is that... I'm not sure the bug report is valid ;)
Seriously, *if* we think it is, must be fixable with a moderate effort.
Here Jason fixed the closely related c++/53721:
http://gcc.gnu.org/ml/gcc-patches/2013-04/msg01445.html
and tweaking a bit more the check would
On Wed, 2013-08-07 at 15:08 -0400, Michael Meissner wrote:
On Tue, Aug 06, 2013 at 11:45:40PM +0200, Oleg Endo wrote:
On Mon, 2013-08-05 at 13:25 -1000, Richard Henderson wrote:
On 08/05/2013 12:32 PM, Oleg Endo wrote:
Thanks, committed as rev 201513.
4.8 also has the same problem.
On Tue, 2013-08-06 at 22:01 -0500, Peter Bergner wrote:
Oleg Endo wrote:
Speaking of GEN_FCN usage in rs6000.c. The recently added HTM builtin
code has one interesting piece:
static rtx
htm_expand_builtin (tree exp, rtx target, bool * expandedp)
{
...
switch (nopnds)
{
On 08/07/2013 03:17 PM, Paolo Carlini wrote:
the issue here is that... I'm not sure the bug report is valid ;)
Seriously, *if* we think it is, must be fixable with a moderate effort.
Here Jason fixed the closely related c++/53721:
http://gcc.gnu.org/ml/gcc-patches/2013-04/msg01445.html
On 07.08.2013 16:59, Jason Merrill wrote:
On 08/07/2013 03:52 AM, Adam Butcher wrote:
But a cleaner way might be to extend the processing
template declaration state from lambda declarator all the way to
the end of the
lambda body. This would match with the scenario that occurs with a
Hello!
You can't read from memory into a mask register in QImode.
This will fail if the memory was the last byte of a page,
and the following page is not mapped.
I expected you to need the following patch, to help spill
and fill QImode values, but I havn't found a test case that
actually
Maciej W. Rozycki ma...@codesourcery.com writes:
BTW, what's the Check for MicroMIPS support. note seen in the
config.host piece of the patch referring to?
No idea, please remove it.
/* Define a function NAME that moves a return value of mode MODE from
FPRs to GPRs. */
-#define
Steve Ellcey sell...@mips.com writes:
Maciej,
When I run two of your new tests in gcc.target/mips (nan-legacy.c and
nans-legacy.c), they are failing because my GCC is putting out
.word 4294967295
instead of
.word -1
like the test is expecting.
I believe they are
Hi,
On 08/07/2013 09:52 PM, Jason Merrill wrote:
The rule that allows use of *this is pretty specific:
Unlike the object expression in other contexts, *this is not required
to be of complete type for purposes of class member access (5.2.5)
outside the member function body.
That doesn't
*const_basic_block;
in target.h. */
typedef int reg_class_t;
+class rtl_opt_pass;
+
+namespace gcc {
+ class context;
+}
+
This break Ada.
../../xgcc -B../../ -c -DIN_GCC -DUSE_LIBUNWIND_EXCEPTIONS -O2 -g -W -Wall \
-iquote /usr/local/gcc/gcc-20130807
Fix error recovery issue with alias:
http://gcc.gnu.org/ml/gcc-patches/2013-06/msg01417.html
Improve debug info for small structures passed by reference:
http://gcc.gnu.org/ml/gcc-patches/2013-06/msg00895.html
Implement static stack checking for the ARM:
Hello!
Attached patch avoids a bunch of -Wcast-qual warnings when building
libstdc++ on alpha-linux-gnu, a _GLIBCXX_LONG_DOUBLE_COMPAT target:
../../../../gcc-svn/trunk/libstdc++-v3/src/c++98/compatibility.cc:521:16:
warning: cast from type ‘const char*’ to type ‘void*’ casts away
qualifiers
PR 58079 is about the do_SUBST assert:
/* Sanity check that we're replacing oldval with a CONST_INT
that is a valid sign-extension for the original mode. */
gcc_assert (INTVAL (newval)
== trunc_int_for_mode (INTVAL (newval), GET_MODE (oldval)));
Hi,
On 08/07/2013 10:48 PM, Uros Bizjak wrote:
2013-08-07 Uros Bizjak ubiz...@gmail.com
* src/c++98/compatibility.cc (_ZTIe): Use const_cast to avoid warning.
(_ZTIPe): Ditto.
(ZTIPKe): Ditto.
The patch was bootstrapped on alpha-linux-gnu, regression test is still running.
Caroline,
When libvtv was checked in, libvtv/autom4te.cache subdirectory was
committed as well. I don't think that this was intended.
- David
+ gcc_patches list
On Wed, Aug 7, 2013 at 2:56 PM, Caroline Tice cmt...@google.com wrote:
No, it was not intended. How do I undo that?
-- Caroline
cmt...@google.com
On Wed, Aug 7, 2013 at 2:54 PM, David Edelsohn dje@gmail.com wrote:
Caroline,
When libvtv was checked in,
.. I removed it.
Thanks,
Paolo.
On Fri, 2 Aug 2013, Ilya Enkovich wrote:
Hi All,
I've updated MPX Wiki page
(http://gcc.gnu.org/wiki/Intel%20MPX%20support%20in%20the%20GCC%20compiler).
I added instrumentation description, programming model description,
differences with other checkers, implementation details.
Thanks. As
Thank you!
-- Caroline
On Wed, Aug 7, 2013 at 3:09 PM, Paolo Carlini paolo.carl...@oracle.com wrote:
.. I removed it.
Thanks,
Paolo.
I would like to add new variations to the mips-mti-elf and mips-mti-linux
targets to support the new NaN format on MIPS (mnan=2008 for IEEE 754-2008).
While I was doing this I noticed that I was handling mips16 and micromips
in a dumb manner. I had them in MULTILIB_OPTIONS as non-exclusive flags
On Wed, Aug 07, 2013 at 02:29:35PM +0200, Dominique Dhumieres wrote:
Revision 201555 breaks boostrap on x86_64-apple-darwin10:
...
Checking multilib configuration for libvtv...
make all-recursive
Making all in testsuite
/bin/sh: line 0: cd: testsuite: No such file or directory
make[4]:
On Fri, 2 Aug 2013, Andrew MacLeod wrote:
The atomic expression patch works on many cases, but falls short with any kind
of complexity on the RHS of an expression which needs to be mapped to an
atomic_load. Whilst parsing, we cant tell, and the rhs tends to become an
expression before we
Here's a patch for the build failure on darwin.
tested x86/linux
-benjamindiff --git a/libvtv/Makefile.am b/libvtv/Makefile.am
index ab3233f..73acfb4 100644
--- a/libvtv/Makefile.am
+++ b/libvtv/Makefile.am
@@ -19,7 +19,11 @@
## along with this library; see the file COPYING3. If not see
##
On Sun, 4 Aug 2013, Andi Kleen wrote:
Richard Biener richard.guent...@gmail.com writes:
The patch fails to add documentation.
That seems like a feature, it's likely not useful for the general
public. More for specialized tools that automatically search
for the best tuning.
If such a
On Wed, Aug 7, 2013 at 12:04 AM, Paolo Carlini paolo.carl...@oracle.com wrote:
By the way, your last patch is fine with me, I like the new *.tcc a lot.
Just allow a day or so for further comments..
Committed.
--
Tim Shen
On Wed, Aug 7, 2013 at 4:54 PM, Joseph S. Myers jos...@codesourcery.com wrote:
On Sun, 4 Aug 2013, Andi Kleen wrote:
Richard Biener richard.guent...@gmail.com writes:
The patch fails to add documentation.
That seems like a feature, it's likely not useful for the general
public. More for
On Wed, 7 Aug 2013, Xinliang David Li wrote:
Index: config/i386/stringop.def
===
--- config/i386/stringop.def (revision 0)
+++ config/i386/stringop.def (revision 0)
@@ -0,0 +1,42 @@
+/* Definitions for option handling for
On Wed, Aug 7, 2013 at 5:23 PM, Joseph S. Myers jos...@codesourcery.com wrote:
On Wed, 7 Aug 2013, Xinliang David Li wrote:
Index: config/i386/stringop.def
===
--- config/i386/stringop.def (revision 0)
+++
Looking at the patch as committed, there seems to be some confusion about
the nature of the --enable-vtable-verify configure option.
It's documented only for libstdc++. Now, I still consider the existence
of separate documentation for libstdc++ configure options to be
unfortunate - I think
On Wed, 7 Aug 2013, Xinliang David Li wrote:
Why the exception? This should only be used on the host, not the target.
Sorry, I copied the boiler-plate header from i386.h -- is it wrong there too?
tm.h gets included in target code because we haven't finished separating
target macros used
78 matches
Mail list logo