Updated patch attached (fixed header, buffer overflow, and warning -->
error problems).
Ok for trunk?
thanks,
David
On Wed, Aug 7, 2013 at 6:04 PM, Joseph S. Myers wrote:
> On Wed, 7 Aug 2013, Xinliang David Li wrote:
>
>> > Why the exception? This should only be used on the host, not the tar
Hi, the attached is a follow up patch. The patch includes the following changes:
1. new option -mno-default -- suggested by H.J. It can be combined
with -mtune-ctrl= to clear the default settings.
2. new option -mdump-tune-features -- this option can be used to dump
all feature names and their def
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 use
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 all
On Wed, Aug 7, 2013 at 5:23 PM, Joseph S. Myers wrote:
> 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
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 fo
On Wed, Aug 7, 2013 at 4:54 PM, Joseph S. Myers wrote:
> On Sun, 4 Aug 2013, Andi Kleen wrote:
>
>> Richard Biener 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 automatical
On Wed, Aug 7, 2013 at 12:04 AM, Paolo Carlini 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 Sun, 4 Aug 2013, Andi Kleen wrote:
> Richard Biener 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 tool is not part of
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 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
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
> mak
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
a
Thank you!
-- Caroline
On Wed, Aug 7, 2013 at 3:09 PM, Paolo Carlini wrote:
> .. 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
.. I removed it.
Thanks,
Paolo.
+ gcc_patches list
On Wed, Aug 7, 2013 at 2:56 PM, Caroline Tice 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 wrote:
>> Caroline,
>>
>> When libvtv was checked in, libvtv/autom4te.cache subdirectory
Caroline,
When libvtv was checked in, libvtv/autom4te.cache subdirectory was
committed as well. I don't think that this was intended.
- David
Hi,
On 08/07/2013 10:48 PM, Uros Bizjak wrote:
2013-08-07 Uros Bizjak
* 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.
OK for mainline if
> 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)));
>
>
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 [-Wc
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:
http://gcc.gnu.org/ml/gcc-patches/20
/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/gcc-20130807/gcc/ada -iquote /usr/local/gcc/gcc-20130807/gcc
> -I/usr/local/
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 seem
"Steve Ellcey " 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 equi
"Maciej W. Rozycki" 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 RET_FUNCTION(N
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
> ac
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
standa
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
and
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 (nop
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 sa
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 al
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 a
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
On Wed, Aug 7, 2013 at 11:34 AM, Andrew Pinski 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
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:
template static void f();
void g() { f(); }
results in "_Z1fIiEvv" under g++, but in "_ZL1fIiEvv" under c
On Wed, Aug 7, 2013 at 10:34 AM, Nathan Sidwell 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
On Wed, Aug 7, 2013 at 11:31 AM, Paul Pluzhnikov wrote:
> On Wed, Aug 7, 2013 at 11:09 AM, Jan Kratochvil
> wrote:
>
>> there should be a new case in:
>> libiberty/testsuite/demangle-expected
>
> Thanks. I updated the patch.
> --
> Paul Pluzhnikov
I think this should also be send to the
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 branc
On Thu, Sep 13, 2012 at 7:46 PM, H.J. Lu 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 may also work cor
ok for google branch.
David
On Tue, Aug 6, 2013 at 3:41 PM, Dehao Chen wrote:
> Patch updated.
>
> http://codereview.appspot.com/12079043
>
> Thanks,
> Dehao
>
>
> On Fri, Aug 2, 2013 at 11:21 AM, Xinliang David Li wrote:
>> More to follow.
>>
>> David
>>
static void
read_profile (void
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 wrote:
> hi Caroline,
>
> A (very) quick look suggests that
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 re
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 -fplugin=
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
> o
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 fo
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
> 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 cha
Fixed the do while formatting. Ok for trunk with this version?
thanks,
David
On Tue, Aug 6, 2013 at 2:42 AM, Jan Hubicka wrote:
>> >>> 2013-08-02 Xinliang David Li
>> >>>
>> >>> * config/i386/stringop.def: New file.
>> >>> * config/i386/stringop.opt: New file.
>> >>>
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'l
Ping.
On Wed, Jul 31, 2013 at 4:34 PM, Easwaran Raman 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 Raman
>
>
On Wed, Aug 7, 2013 at 8:37 AM, Jan Hubicka wrote:
>> Ping?
>>
>> David
>>
>> On Sat, Aug 3, 2013 at 12:36 PM, Xinliang David Li
>> wrote:
>> > Hi, GCC/i386 currently has about 73 boolean parameters/knobs (defined
>> > in ix86_tune_features[], indexed by ix86_tune_indices) to perform
>> > micro-
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 expre
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 definiti
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:
There
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 be
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.
> Ping?
>
> David
>
> On Sat, Aug 3, 2013 at 12:36 PM, Xinliang David Li 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 ha
Ping?
David
On Sat, Aug 3, 2013 at 12:36 PM, Xinliang David Li 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 hard
> coded (fixe
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 th
> 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 arm-n
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,
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 twi
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
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
* configure.ac (lm32*-*-*): Use SJLJ exceptions.
--
Joel Sherrill, Ph.D. Director of Research & Development
joel.sher
> 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 build&host x86_64. Was able to build a cc1
> and step through the changed code in the debugger, though am getting
> "cc1: erro
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
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.
Thank
> 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 advanc
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]: *** [all-stage1
Hi,
I'm applying this mainline and 4_8-branch. Tested x86_64-linux.
Thanks,
Paolo.
//
2013-08-07 Paolo Carlini
PR libstdc++/58098
* include/bits/random.h (cauchy_distribution<>::min,
extreme_value_distribution<>::min, normal_distribution<>::min,
Richard Sandiford 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 (INTVAL (newval), GET
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
On Tue, Aug 6, 2013 at 6:42 PM, Ian Lance Taylor 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
>
> * go-gcc.cc (Gcc_backend::
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
> >+
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,
P
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 com
> the option is designed for purpose like this.
That's great, thanks!
Michael
> David
On 6 August 2013 20:42, Xinliang David Li wrote:
> Corrected two small problems reported by the style checker (The
> warnings about the EnumValue for options in stringopt.opt are not
> valid).
>
> On Tue, Aug
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 t
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/gcc \
-iquote . -iquote ..
80 matches
Mail list logo