On 07/25/2013 03:35 AM, Jason Merrill wrote:
You need to have tests for the other string prefixes, since you're
changing the code for them.
Oh yes, something like this?
(believe me, I had it, but then the last minute I simplified the
testcase because the last lines didn't seem to exercise any
You need to have tests for the other string prefixes, since you're
changing the code for them.
Jason
> On 23 July 2013 23:32:27 pcha...@cs.wisc.edu wrote:
>> Hi,
>>
>> The problem appears in revision 201034 in version 4.9. I attached a
>> one-line patch that fixes it. I also reported this problem
>> at http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57800
>>
>> Bootstrap and regression-tested on x86_
On Wed, 2013-07-24 at 19:10 -0400, Diego Novillo wrote:
> On Wed, Jul 24, 2013 at 6:56 PM, Diego Novillo wrote:
> > Could you please add a description of what this does?
>
> Sorry. You did, but in a previous message that I had managed to miss.
> Maybe include a reference to it in future posting
On Wed, 2013-07-24 at 18:55 -0400, Diego Novillo wrote:
> On Wed, Jul 24, 2013 at 11:09 AM, David Malcolm wrote:
> > gcc/
> >
> > Introduce context class.
> >
> > * Makefile.in (CONTEXT_H): New.
> > (OBJS): Add context.o.
> > (toplev.o): Add CONTEXT_H to dependencie
On Wed, Jul 24, 2013 at 6:56 PM, Diego Novillo wrote:
> Could you please add a description of what this does?
Sorry. You did, but in a previous message that I had managed to miss.
Maybe include a reference to it in future postings?
Diego.
On Wed, 24 Jul 2013, Ilya Enkovich wrote:
> Well, this patch does not introduce any changes on user-visible level.
> It just adds MPX instructions support to i386 target. Usually each new
> x86 instruction has corresponding builtin function and therefore is
> provided with a testcase. But MPX inst
Could you please add a description of what this does?
On Wed, Jul 24, 2013 at 11:09 AM, David Malcolm wrote:
> gcc/
> * Makefile.in (PIPELINE_H): New.
> (lto-cgraph.o): Depend on CONTEXT_H and PIPELINE_H.
> (passes.o): Likewise.
> (statistics.o): Likewise.
>
On Wed, Jul 24, 2013 at 11:09 AM, David Malcolm wrote:
> gcc/
>
> Introduce context class.
>
> * Makefile.in (CONTEXT_H): New.
> (OBJS): Add context.o.
> (toplev.o): Add CONTEXT_H to dependencies.
> (context.o): New.
>
> * toplev.c (general_init): Cr
I merged gcc 4.8 branch revision 201227 to the gccgo branch.
Ian
When a function created by cgo or SWIG returns an interface type and at
runtime returns a value that requires allocation when converting to
interface, it will call the memory allocation routine while appearing to
be in a system call. In particular there will be no memory cache
allocated. This pat
Hi,
submitter ran a static analysis tool on parser.c and found in the main
switch of cp_parser_operator some broken cases having to do with user
defined literals. When I looked into it I found a rather convoluted
logic which tried to possibly emit *both* the error message for and
invalid enco
Can you collect some number on ggc_memory increase with this change
in profile_gen build -- the value is recorded gcov_module_info object
in coverage.c.
thanks,
David
On Wed, Jul 24, 2013 at 3:14 PM, Sriraman Tallam wrote:
> The following test-case fails in LIPO mode during profile-use build:
The following test-case fails in LIPO mode during profile-use build:
main.cc
__attribute__((target("sse4.2")))
unsigned int problem_aux (unsigned int A, unsigned int B);
int main (int argc, char *argv[])
{
return problem_aux (0, 0);
}
aux.cc
-
__attribute__((target("sse4.
On Wed, Jul 24, 2013 at 11:53:09PM +0200, Janus Weil wrote:
> Hi all,
>
> here is a straightforward patch for an ICE-on-invalid problem, which
> basically adds some checks for 'class_ok'.
>
> Regtested on x86_64-unknown-linux-gnu. Ok for trunk?
>
OK.
--
Steve
Hi all,
here is a straightforward patch for an ICE-on-invalid problem, which
basically adds some checks for 'class_ok'.
Regtested on x86_64-unknown-linux-gnu. Ok for trunk?
Cheers,
Janus
2013-07-24 Janus Weil
PR fortran/57639
* interface.c (compare_parameter): Check for class_ok.
On Wed, Jul 24, 2013 at 5:05 PM, Bill Schmidt
wrote:
> In little endian mode, we need to reverse the order of operands for
> vector pack operations. This corrects 20 failures in the test suite in
> LE mode.
>
> Patch based on work by Anton Blanchard; I simplified it and generalized
> it to all th
On 07/19/2013 05:32 PM, DJ Delorie wrote:
Every pattern that is using (subreg:SI (thing:PSI)) needs to be
explained on this list and given an explicit clearance. It really looks
like you're just papering over problems elsewhere.
Most of them are just optimizations, but the problem with reload
In little endian mode, we need to reverse the order of operands for
vector pack operations. This corrects 20 failures in the test suite in
LE mode.
Patch based on work by Anton Blanchard; I simplified it and generalized
it to all the vector pack insns.
Bootstrapped and tested on powerpc64-unknow
On 07/24/2013 06:07 AM, Joern Rennecke wrote:
The problem is fixed by simply moving the code that skips insns that don't
set a part of the return value below the code that checks if the insn needs
a non-MODE_EXIT mode setting.
bootstrapped/regtested on i686-pc-linux-gnu
2013-07-24 Joern Renne
I merged gcc 4.8 branch revision 201223 to the gccgo branch.
Ian
When a Go global variable has zero size, we fudge it to make it larger
to avoid problems with the linker. This turned out to cause a bug with
a struct with a zero-sized field, since code still expected to be able
to access the field. This patch changes the zero size handling to
recreate such stru
2013/7/24 Joseph S. Myers :
> On Wed, 24 Jul 2013, Ilya Enkovich wrote:
>
>> Here is a patch which adds support for new instructions from Intel
>> Memory Protection Extensions (MPX) ISA [1]
>>
>> This patch introduces bound type, modes, registers and all MPX instructions.
>>
>> Control transfer ins
On Wed, Jul 24, 2013 at 4:49 AM, Rainer Orth
wrote:
> Ian Lance Taylor writes:
>
>> On Thu, Jul 18, 2013 at 6:59 AM, Rainer Orth
>> wrote:
>>> Ian Lance Taylor writes:
>>>
I have committed a large patch to update libgo to the library that was
part of the Go 1.1.1 release. As usual, I
On Wed, Jul 24, 2013 at 4:49 AM, Rainer Orth
wrote:
> Ian Lance Taylor writes:
>
>> On Thu, Jul 18, 2013 at 6:59 AM, Rainer Orth
>> wrote:
>>> Ian Lance Taylor writes:
>>>
I have committed a large patch to update libgo to the library that was
part of the Go 1.1.1 release. As usual, I
Hello,
As discussed here, the current behavior of -ftls-model is intended:
http://gcc.gnu.org/ml/gcc/2013-07/msg00248.html
(in short: unlike the attribute, -ftls-model sets a "most general" model to be
used, but the compiler can use a less general model than specified, making it
impossible to set
Hi, Joseph,
Sorry for that we have to zip this patch. It seems that the plaintext
revised patch is too large to be sent to mailing list due to lots of
copyright information.
And I am really sorry for that you probably kept receiving the same mail.
Somehow the this [PATCH 3/6] mail just didn't a
On 24 July 2013 18:12:00 "Bernhard Reutner-Fischer"
wrote:
On 23 July 2013 23:32:27 pcha...@cs.wisc.edu wrote:
> Hi,
>
> The problem appears in revision 201034 in version 4.9. I attached a
> one-line patch that fixes it. I also reported this problem
> at http://gcc.gnu.org/bugzilla/show_bug.cgi
Thanks, checked in!
On Wed, Jul 24, 2013 at 8:06 PM, Gerald Pfeifer wrote:
> On Wed, 24 Jul 2013, Kirill Yukhin wrote:
>> Patch adds mention of avx512 and mpx branches in svn.html.
>>
>> Ok to install?
>
> Yes, thanks!
>
> Gerald
On 7/12/13 1:35 AM, Chung-Ju Wu wrote:
On 7/10/13 8:11 AM, Joseph S. Myers wrote:
Don't you have built-in functions to document as well?
You are right.
We did forget to describe built-in functions in the documentation.
Thanks for pointing out our carelessness. :)
Use "@:" (or a comma) after
On Thu, Jul 25, 2013 at 12:13 AM, Paolo Carlini
wrote:
> Thanks. I added a few words to 57513 too. It would be nice to figure out
> which specific change fixed the link-time issue (provided a definition for
> the default constructor?)
Comment added.
--
Tim Shen
Hi, Joseph,
Sorry for the late revised patch.
We have completed all of it based on your review comments.
On 7/10/13 8:08 AM, Joseph S. Myers wrote:
On Mon, 8 Jul 2013, Chung-Ju Wu wrote:
* gcc.dg/ucnid-11.c: Skip for nds32*-*-*.
* gcc.dg/ucnid-2.c: Skip for nds32*-*-*.
On 07/24/2013 05:18 PM, Jonathan Wakely wrote:
Should I register a new bugzilla account to comment on these two
bugs(53622 and 57173), or any other one will do this?
I've added a comment to the bugs.
With svn commit rights you should be able to receive email at @gcc.gnu.org and if you use that a
On Wed, 24 Jul 2013, Chung-Ju Wu wrote:
> We removed the fancy comment formatting as you suggested.
> However, for the comments like:
>
> /* = */
> /* Title of section */
> /* = */
>
> /* Title of subsection 1 */
>
> /* Title of subsection 2 */
>
> I wis
On 23 July 2013 23:32:27 pcha...@cs.wisc.edu wrote:
Hi,
The problem appears in revision 201034 in version 4.9. I attached a
one-line patch that fixes it. I also reported this problem
at http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57800
Bootstrap and regression-tested on x86_64-linux.
In metho
On Wed, 24 Jul 2013, Kirill Yukhin wrote:
> Patch adds mention of avx512 and mpx branches in svn.html.
>
> Ok to install?
Yes, thanks!
Gerald
On Wed, 24 Jul 2013, Ilya Enkovich wrote:
> Here is a patch which adds support for new instructions from Intel
> Memory Protection Extensions (MPX) ISA [1]
>
> This patch introduces bound type, modes, registers and all MPX instructions.
>
> Control transfer instructions are modified to support '
Hi, Joseph,
Sorry for the late revised patch.
We have completed all of it based on your review comments.
On 7/10/13 6:56 AM, Joseph S. Myers wrote:
On Mon, 8 Jul 2013, Chung-Ju Wu wrote:
+ nds32*-*-*)
+ supported_defaults="arch nds32_lib"
+
+ # process --with
Hi,
On Tue, Jul 23, 2013 at 11:43:04AM -0400, David Malcolm wrote:
> On Tue, 2013-07-23 at 16:46 +0200, Martin Jambor wrote:
> > Hi,
> >
> > On Mon, Jul 22, 2013 at 03:22:33PM -0400, David Malcolm wrote:
> > > On Mon, 2013-07-22 at 20:25 +0200, Martin Jambor wrote:
> > > > On Wed, Jul 17, 2013 at
Hi,
Here is a patch which adds support for new instructions from Intel
Memory Protection Extensions (MPX) ISA [1]
This patch introduces bound type, modes, registers and all MPX instructions.
Control transfer instructions are modified to support 'bnd' prefix
used by Intel MPX.
Bootstrap and chec
On 24 July 2013 15:50, Tim Shen wrote:
>
> Committed.
Thanks!
> Should I register a new bugzilla account to comment on these two
> bugs(53622 and 57173), or any other one will do this?
I've added a comment to the bugs.
With svn commit rights you should be able to receive email at @gcc.gnu.org a
On 07/24/2013 04:50 PM, Tim Shen wrote:
On Wed, Jul 24, 2013 at 5:18 PM, Paolo Carlini wrote:
Then we should also add something (by hand, unfortunately, as I explained)
to the audit trails of those bugs, because we want the original submitters
to double check the fixes with their own code, get
gcc/
* Makefile.in (PIPELINE_H): New.
(lto-cgraph.o): Depend on CONTEXT_H and PIPELINE_H.
(passes.o): Likewise.
(statistics.o): Likewise.
(cgraphunit.o): Likewise.
(context.o): Depend on PIPELINE_H.
* pipeline.h: New.
* cgraphunit.c
The following pair of patches is aimed at eliminating global variables
from gcc's internals, as per my Cauldron talk on state-removal [1]
Patch 1 introduces a gcc::context class, along with a singleton
instance. The name of the instance is chosen so as to be easy to
type in the debugger:
gcc:
gcc/
Introduce context class.
* Makefile.in (CONTEXT_H): New.
(OBJS): Add context.o.
(toplev.o): Add CONTEXT_H to dependencies.
(context.o): New.
* toplev.c (general_init): Create the singleton gcc::context
instance.
* context.c: N
gcc.target/ppc-vector-memcpy.c testcase currently fails and has been
failing since early in the GCC 4.6 development cycle. A change by
Eric Botcazou in April 2008 to the handling of constants reduced the
extra alignment of initializers matching the alignment of the
variable.
The testcases origina
On Wed, Jul 24, 2013 at 5:18 PM, Paolo Carlini wrote:
> Then we should also add something (by hand, unfortunately, as I explained)
> to the audit trails of those bugs, because we want the original submitters
> to double check the fixes with their own code, get as much feedback as
> possible while
Hi,
Patch adds mention of avx512 and mpx branches in svn.html.
Ok to install?
Index: htdocs/svn.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/svn.html,v
retrieving revision 1.185
diff -p -r1.185 svn.html
*** htdocs/svn.html 27 May 2
On Wed, Jul 24, 2013 at 4:49 AM, Rainer Orth
wrote:
> Ian Lance Taylor writes:
>
>> On Thu, Jul 18, 2013 at 6:59 AM, Rainer Orth
>> wrote:
>>> Ian Lance Taylor writes:
>>>
I have committed a large patch to update libgo to the library that was
part of the Go 1.1.1 release. As usual, I
On Wed, Jul 24, 2013 at 2:24 AM, Uros Bizjak wrote:
> On Wed, Jul 24, 2013 at 9:09 AM, Uros Bizjak wrote:
>
The test was re-run with GOTESTFLAGS=--keep. When running the
resulting a.out with "strace -f -o strace-x86_64 ./a.out" from the
saved test directory, the test behaved in the
On Tue, Jul 23, 2013 at 11:19 PM, Bill Schmidt
wrote:
> In order to use vperm for aligning loads and stores in little endian
> mode, we need to reverse the order of the input operands and use lvsl
> instead of lvsr. This corrects 32 regressions in the test suite when
> run in LE mode.
>
> Bootstr
The problem is fixed by simply moving the code that skips insns that don't
set a part of the return value below the code that checks if the insn needs
a non-MODE_EXIT mode setting.
bootstrapped/regtested on i686-pc-linux-gnu
2013-07-24 Joern Rennecke
PR rtl-optimization/57968
Ian Lance Taylor writes:
> On Thu, Jul 18, 2013 at 6:59 AM, Rainer Orth
> wrote:
>> Ian Lance Taylor writes:
>>
>>> I have committed a large patch to update libgo to the library that was
>>> part of the Go 1.1.1 release. As usual, I'm not including the entire
>>> patch in this e-mail message,
ping.
Thanks.
bin
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org]
On
> Behalf Of Bin Cheng
> Sent: Tuesday, July 09, 2013 4:08 PM
> To: gcc-patches@gcc.gnu.org
> Subject: [PATCH GCC/ARM]Disable "-fira-hoist-pressure" on Thumb2
>
> Hi,
> F
Ping this patch:
> > [PATCH ARM]Extend thumb1_reorg to save more comparison instructions
> > http://gcc.gnu.org/ml/gcc-patches/2013-04/msg01057.html
Thanks.
bin
> -Original Message-
> From: Jeff Law [mailto:l...@redhat.com]
> Sent: Saturday, July 20, 2013 1:34 AM
> To: Bin Cheng
> Cc: gc
On Wed, Jul 24, 2013 at 9:09 AM, Uros Bizjak wrote:
>>> The test was re-run with GOTESTFLAGS=--keep. When running the
>>> resulting a.out with "strace -f -o strace-x86_64 ./a.out" from the
>>> saved test directory, the test behaved in the same way as on alpha -
>>> it hever finished. I have attac
On 07/23/2013 05:31 PM, Tim Shen wrote:
Revised version, with _M_dfs_* moved to the class definition, PR
testcases, and some comments on _M_dfs().
I nobody has further comments, I think this is Ok to go in now. Let's
make sure we don't forget the various comments about optimization,
pre-scannin
Hi,
On 07/23/2013 12:45 PM, Chung-Ju Wu wrote:
2013/7/23 Paolo Carlini :
... in the future, when we manage to actually avoid including from
I think we can avoid including it from if we play a bit
with void*. Should be safe aliasing-wise. Something like the below,
completely untested.
I test
Hi,
the hardware transactional execution instructions in libitm are only
enabled if actually supported by the assembler. So the option which
enables the instructions in the assembler should only be set if we are
already in a code path where we know the assembler can do it.
With the attached patc
On 24/07/13 03:36, Terry Guo wrote:
Hi there,
This patch intends to update thumb1_size_rtx_costs function to correctly
handle those RTXs defined by RTL expansion pass. Thus the GIMPLE
multiplication will be expanded to single mul instruction instead of a bunch
of shift/add/sub instructions which
Hi Joey,
This patch is to define multilibs for recently created embedded-4_8-branch.
Is it OK to commit?
BR,
Terry
2013-07-24 Terry Guo
* configure.ac (with_multilib_list): Export its value.
* Makefile.in (with_multilib_list): Import it from configure files.
* configu
On 07/24/2013 03:24 AM, Gabriel Dos Reis wrote:
Do you envision this to be useful for C++ too?
There are many bits here. 'const' in C++ does not necessary prevent
race conditions with the variable's type has to run non-atomic constructors,
or if the object has data members declared mutable,
On Wed, Jul 24, 2013 at 12:23 AM, Ian Lance Taylor wrote:
> On Mon, Jul 22, 2013 at 6:38 AM, Uros Bizjak wrote:
>>
>>> I have committed a large patch to update libgo to the library that was
>>> part of the Go 1.1.1 release. As usual, I'm not including the entire
>>> patch in this e-m
63 matches
Mail list logo
