mary ===
# of expected passes81909
# of unexpected failures33
# of unexpected successes 20
# of expected failures 271
# of unsupported tests 1995
/home/user/gcc-build/gcc/xgcc version 4.7.0 20111017 (experimental) (GCC)
=== g++ tests ===
Ru
There must be a style lint for gcc -- but I have not used it ..
http://codereview.appspot.com/5272048/diff/2001/tree-asan.c
File tree-asan.c (right):
http://codereview.appspot.com/5272048/diff/2001/tree-asan.c#newcode79
tree-asan.c:79: (All I need is to traverse *all* memory accesses and
instru
On Mon, 17 Oct 2011, Dodji Seketeli wrote:
>> this looks like it's causing the following bootstrap failure for me
>> on i386-unknown-freebsd9.0?
> Yes this is http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50760, and I am
> testing the patch attached to the bug at the moment. Just curious, does
> tha
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
tree-ssa-sink.c could benefit from a little TLC in its code to select
statement's final location.
For those not familiar with tree-ssa-sink, it attempts to sink
statements down in the CFG to points where they're less likely to
execute. Ideally we mo
This is now in, thanks everybody for playing! Attached is the final
diff.
tested x86_64/linux
-benjamindiff --git a/gcc/ChangeLog b/gcc/ChangeLog
index ddc5149..6c73404 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2011-10-17 Michael Spertus
+
+ * gcc/c-family/c-common.c (c_
On 10/12/2011 03:15 PM, Diego Novillo wrote:
+static void
+cp_debug_print_unparsed_function (FILE *file, cp_unparsed_functions_entry *uf)
You probably want to handle uf->nsdmis here too now.
Otherwise looks good.
Jason
On 10/17/2011 06:33 PM, Dodji Seketeli wrote:
OK if this appears to fix the raised issues and passes bootstraps on the
i686 target?
If you have a patch like this that fixes a major regression, go ahead
and check it in without waiting for approval; we can adjust it as
necessary after build is
On Fri, Oct 14, 2011 at 6:53 AM, Julian Brown wrote:
> On Wed, 28 Sep 2011 14:33:17 +0100
> Ramana Radhakrishnan wrote:
>
>> On 6 May 2011 14:13, Julian Brown wrote:
>> > Hi,
>> >
>> > This is the second of two patches to add unaligned-access support to
>> > the ARM backend. It builds on the fir
This is Richard's patch to add vec_perm patterns for VIS2, I verified
that the patterns do get used in some of the slp test cases and that
such test cases pass.
I only had to make a minor adjustment due to a conflict with the
vector segregation patch, namely the mode iterator had to be changed
fr
On 13/10/2011, at 12:58 AM, Richard Guenther wrote:
> On Wed, Oct 12, 2011 at 8:50 AM, Maxim Kuvyrkov
> wrote:
>> The following patch adds new knob to make GCC perform several iterations of
>> early optimizations and inlining.
>>
>> This is for dont-care-about-compile-time-optimize-all-you-can
A small patch from Richard to fix the edge predicates, committed
to trunk.
* config/sparc/sparc.md: Use register_or_zero_operand where rJ
is the constraint.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@180113
138bc75d-0d04-0410-961f-82ee72b054a4
---
gcc/ChangeLog
From: Eric Botcazou
Date: Tue, 18 Oct 2011 00:57:18 +0200
>> I would suggest we start with my patch, get the int<-->float move
>> instructions working reasonably, and then re-add vector-only cases for
>> the scenerios you describe above, making sure that we don't end up with
>> silly code generat
> I would suggest we start with my patch, get the int<-->float move
> instructions working reasonably, and then re-add vector-only cases for
> the scenerios you describe above, making sure that we don't end up with
> silly code generation scenerios like those I've just described.
This sounds like
"H.J. Lu" writes:
> There are at least 2 bootstrap problems:
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50760
The patch below should address those issues.
The first two hunks are to fix bootstrap for targets that have
!NO_IMPLICIT_EXTERN_C. To test it, I built the compiler with
--target=a
http://codereview.appspot.com/5272048/diff/2001/tree-asan.c
File tree-asan.c (right):
http://codereview.appspot.com/5272048/diff/2001/tree-asan.c#newcode79
tree-asan.c:79: (All I need is to traverse *all* memory accesses and
instrument them).
Discard 2) -- it is not correct. What Asan needs is
From: Eric Botcazou
Date: Tue, 18 Oct 2011 00:09:55 +0200
> I think that the original motivation for the previous design was the 32-bit
> vector ABI, where the arguments are passed in integer registers. So for:
>
> typedef char vec8 __attribute__((vector_size(8)));
>
> extern vec8 foo (vec8)
fasan option also needs to be documented in doc/invoke.texi.
http://codereview.appspot.com/5272048/diff/2001/tree-asan.c
File tree-asan.c (right):
http://codereview.appspot.com/5272048/diff/2001/tree-asan.c#newcode54
tree-asan.c:54: ShadowValue = (char*)ShadowAddr;
*(char*) ShadowAddr;
http://
Nicola Pero writes:
> > (I don't have svn write access so I'll need someone else to commit it if
> > it's approved.)
>
> I can apply it for you. But ... do you have a copyright assignment in place
> for contributions to GCC ? The patch looks small and trivial enough that
> I think I can ap
> This is an implementation of the changes I spoke about the other
> week. These changes segregate the vector vs. non-vector mode
> handling in the sparc backend.
I think that the original motivation for the previous design was the 32-bit
vector ABI, where the arguments are passed in integer reg
Hi all,
here is another patch for PR47023, which takes care of comment #1,
i.e. rejecting polymorphic variables in a C-binding context. It also
adapts two functions (verify_c_interop,verify_c_interop_param) to the
gfortran naming convention of prefixing public routines with 'gfc_',
and fixes an er
On Mon, Oct 17, 2011 at 19:03, Tobias Burnus wrote:
> Hi Janne,
>
> On 10/17/2011 05:30 PM, Janne Blomqvist wrote:
>>
>> On Mon, Oct 17, 2011 at 15:49, Tobias Burnus wrote:
>>>
>>> This patch adds a call to _commit() on _WIN32 for the FLUSH subroutine
>>> and
>>> the FLUSH statement. It removes t
On 10/17/2011 12:21 PM, Jakub Jelinek wrote:
> 2011-10-17 Jakub Jelinek
>
> * gcc.c-torture/execute/vshuf-v16hi.c: New test.
> * gcc.dg/torture/vshuf-16.inc: New file.
> * gcc.dg/torture/vshuf-2.inc: New file.
> * gcc.dg/torture/vshuf-32.inc: New file.
> * gcc.dg/t
> 2011-10-17 Richard Guenther
>
> PR middle-end/50716
> * expr.c (get_object_or_type_alignment): New function.
> (expand_assignment): Use it.
> (expand_expr_real_1): Likewise.
Maybe move it to builtins.c alongside the other get_*_alignment functions.
> Index: gcc/expr.c
This patch changes Technical Report 29113 to Technical Specification
29113 as ISO/SC22 decided that the TR29113 draft should be submitted as
TS; it reflects the recently committed patch to gfortran itself. [1]
Tobias
[1] http://gcc.gnu.org/ml/fortran/2011-10/msg00088.html
Index: changes.html
=
Hi!
This patch adds some further __builtin_shuffle tests with constant masks.
As the full tests are quite expensive (even when I've cut it down from 512
generated masks to 128), I've split the masks into TESTS (currently 16, plus
I plan to add a few permutation masks based on gcov coverage testing
Gerald Pfeifer writes:
> Hi Dodji,
>
> On Mon, 17 Oct 2011, Dodji Seketeli wrote:
>> This is the first instalment of a set which goal is to track locations
>> of tokens across macro expansions. Tom Tromey did the original work
>> and attached the patch to PR preprocessor/7263. This opus is a
>>
Hi,
vect-114.c doesn't have misaligned accesses, so there is no need in
vect_hw_misalign.
Tested on powerpc64-suse-linux.
Committed.
Ira
testsuite/ChangeLog:
PR tree-optimization/50746
* gcc.dg/vect/vect-114.c: Remove vect_hw_misalign.
Index: testsuite/gcc.dg/vect/vect-114.c
=
Here we were failing to check whether the thing being captured was an
automatic variable, as required by the standard.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 2e585b32e7c969c1dc2ae7b2a62978b95ed4bf17
Author: Jason Merrill
Date: Sun Oct 16 01:15:09 2011 -0400
PR c++/50736
Hi Dodji,
On Mon, 17 Oct 2011, Dodji Seketeli wrote:
> This is the first instalment of a set which goal is to track locations
> of tokens across macro expansions. Tom Tromey did the original work
> and attached the patch to PR preprocessor/7263. This opus is a
> derivative of that original work.
Agh. Sorry about this Rainer. Thanks for the patch.
-benjamin
On Mon, Oct 17, 2011 at 12:31 PM, Jason Merrill wrote:
> On 10/17/2011 07:48 AM, Paolo Carlini wrote:
>>
>> And please help
>> re-assessing the situation wrt the other front-ends *today* not in the
>> stone age.
>
> EDG always wraps diagnostics at ~80 columns.
I did not mention the name of the co
On 10/17/11 19:43, Eric Botcazou wrote:
>> Yes. However, in the case that caused the PR, this was attempted with
>> reload_completed == 0, so we cannot generate return patterns anyway and
>> must fail.
>
> OK, but we clean up the CFG after reload is completed (e.g. just before
> emitting the prol
> Yes. However, in the case that caused the PR, this was attempted with
> reload_completed == 0, so we cannot generate return patterns anyway and
> must fail.
OK, but we clean up the CFG after reload is completed (e.g. just before
emitting the prologue) so in this case we can generate return patt
On 10/16/2011 05:01 PM, Fabien Chêne wrote:
[...]
Does the attached testcase checked what you mention ?
... with the testcase attached.
Yes, that's what I had in mind.
Jason
OK.
Jason
OK.
Jason
On 10/17/2011 07:31 PM, Jason Merrill wrote:
clang wraps diagnostics at $COLUMNS when stderr is going to a
terminal, and doesn't wrap otherwise.
The clang behavior seems like the right way to go.
Thanks Jason. I'll see how to implement this.
Paolo.
Hi,
exactly like the recently fixed c++/17212. Tested x86_64-linux.
Ok for mainline?
Thanks,
Paolo.
/
/gcc
2011-10-17 Paolo Carlini
PR c++/50757
* c-family/c.opt ([Wnonnull]): Add C++ and Objective-C++.
* doc/invoke.texi: Update.
/testsuite
On 10/17/2011 07:48 AM, Paolo Carlini wrote:
And please help
re-assessing the situation wrt the other front-ends *today* not in the
stone age.
EDG always wraps diagnostics at ~80 columns.
clang wraps diagnostics at $COLUMNS when stderr is going to a terminal,
and doesn't wrap otherwise.
The
I plan to check in the following patch as obvious after it passed
testing.
cygwin has a MADV_DONTNEED, but does not use mmap. The ifdefs for
madvise assumed this wouldn't happen and it broke the cygwin build.
Just don't set USING_MADVISE when USING_MMAP is not set. Thanks
to Kai Titz for testin
On Mon, Oct 17, 2011 at 6:41 AM, Dodji Seketeli wrote:
> Finally here is what I am checking in, which passes bootstrap with
> --disable-checking --enable-languages=all,ada -- modulo this other bug
> that breaks bootstrap as well
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50709.
>
> It's been OK
On Mon, Oct 17, 2011 at 12:22:03PM +0200, Tobias Burnus wrote:
> On 10/15/2011 11:21 PM, Steve Kargl wrote:
> >OK for trunk?
> >
> >2011-10-15 Steven G. Kargl
> >
> > * gfortran.dg/ishft_3.f90: Update test.
>
> I am not so happy with complete test replacements. How about adding it
> as new
2011/10/17 Georg-Johann Lay :
> This is more code clean-up for insn attributes.
>
> It removes "mcu_have_movw", "mcu_mega" and defines "enabled" and "isa"
> attributes instead.
>
> The "isa" attribute which triggers "enabled" is a replacement for
> AVR_HAVE_MOVW
> assembler dialect. We don't actu
Hi Julian,
There are a couple of minor formatting nits.
>+static int
>+arm_movmemqi_unaligned (rtx *operands)
>+ /* Inlined memcpy using ldr/str/ldrh/strh can be quite big: try to limit
>+ size of code if optimizing for size. We'll use ldm/stm if
>src_aligned
>+ or ds
>> Be quick, though; 4.6.2 was due for release now-ish -
>
> I know. I'll try to do it today.
Alrighty. Just committed the patch to the 4.6 branch as r180099.
Seems we're down to three open Fortran front-end regressions for the
upcoming 4.6.2 release:
* http://gcc.gnu.org/bugzilla/show_bug.cgi
Hi Janne,
On 10/17/2011 05:30 PM, Janne Blomqvist wrote:
On Mon, Oct 17, 2011 at 15:49, Tobias Burnus wrote:
This patch adds a call to _commit() on _WIN32 for the FLUSH subroutine and
the FLUSH statement. It removes the _commit from gfortran's buf_flush.
Like I argued in this message
http://
On Mon, Oct 17, 2011 at 15:49, Tobias Burnus wrote:
> This patch adds a call to _commit() on _WIN32 for the FLUSH subroutine and
> the FLUSH statement. It removes the _commit from gfortran's buf_flush.
Like I argued in this message
http://gcc.gnu.org/ml/fortran/2011-10/msg00094.html ,
I think t
On 10/16/2011 01:45 PM, Uros Bizjak wrote:
> libgcc/ChangeLog:
>
> 2011-10-16 Uros Bizjak
> Eric Botcazou
>
> PR target/50737
> * config/alpha/linux-unwind.h (alpha_fallback_frame_state): Set
> fs->signal_frame to 1.
>
> libjava/ChangeLog:
>
> 2011-10-16 Uros B
On 10/17/2011 05:00 PM, Rainer Orth wrote:
As described in the PR, after Benjamin's patch
2011-10-10 Benjamin Kosnik
PR libstdc++/49818
* config/abi/pre/gnu.ver (CXXABI_1.3.6): Add symbols.
__cxa_get_exception_ptr is now present in both CXXABI_1.3.1 and
CXXABI_1.3.6, which b
As described in the PR, after Benjamin's patch
2011-10-10 Benjamin Kosnik
PR libstdc++/49818
* config/abi/pre/gnu.ver (CXXABI_1.3.6): Add symbols.
__cxa_get_exception_ptr is now present in both CXXABI_1.3.1 and
CXXABI_1.3.6, which breaks Solaris bootstrap with Sun ld:
ld: fat
Thanks, guys, could anybody please commit that?
K
On Mon, Oct 17, 2011 at 6:33 PM, Jakub Jelinek wrote:
> On Mon, Oct 17, 2011 at 06:27:04PM +0400, Kirill Yukhin wrote:
>> Thanks for inputs, Jakub!
>>
>> I am attaching updated patch.
>>
>> Updated testsuite/ChangeLog entry:
>> 2011-10-17 Kirill
paul_kon...@dell.com schrieb:
>> There is no real post morten debugging on AVR as there is nothing like
>> core dump.
>
> But if there is any kind of debugging at all, you might use the debugger to
> put a breakpoint in abort(), and if so, having that invoked via jmp means
> you don't know who cal
I'm checking this in as obvious.
Sergio pointed out, via this patch, that gcc/configure didn't properly
emit whether sys/std.h was discovered.
Tested by re-running configure and examining the output.
Tom
2011-10-17 Sergio Durigan Junior
* configure.ac: Display `yes' if the SystemTap
On Mon, Oct 17, 2011 at 3:36 PM, Kai Tietz wrote:
> Sure,
>
> Is simplier and also handles (A T[-IF] (B T-IF C) -> (A T B) T-IF C
> case, which can happen by framing in conditions.
>
> @@ -8380,13 +8400,65 @@ fold_truth_andor (location_t loc, enum t
> lhs is another similar operation, try to
>There is no real post morten debugging on AVR as there is nothing like core
>dump.
But if there is any kind of debugging at all, you might use the debugger to put
a breakpoint in abort(), and if so, having that invoked via jmp means you don't
know who called it. So you'd want a way to suppres
On Mon, Oct 17, 2011 at 06:27:04PM +0400, Kirill Yukhin wrote:
> Thanks for inputs, Jakub!
>
> I am attaching updated patch.
>
> Updated testsuite/ChangeLog entry:
> 2011-10-17 Kirill Yukhin
>
> * gcc.target/i386/avx2-vpop-check.h: New header.
> * gcc.target/i386/avx2-vpaddd-3
This is more code clean-up for insn attributes.
It removes "mcu_have_movw", "mcu_mega" and defines "enabled" and "isa"
attributes instead.
The "isa" attribute which triggers "enabled" is a replacement for AVR_HAVE_MOVW
assembler dialect. We don't actually support assembler dialects but different
On Fri, 14 Oct 2011, Richard Guenther wrote:
>
> This follows up Michas testcase where we fail to handle the
> conservatively propagated restrict tags properly. The following
> patch simplifies handling of restrict in the oracle and thus
> only excludes NONLOCAL (as designed), but not ESCAPED fr
Thanks for inputs, Jakub!
I am attaching updated patch.
Updated testsuite/ChangeLog entry:
2011-10-17 Kirill Yukhin
* gcc.target/i386/avx2-vpop-check.h: New header.
* gcc.target/i386/avx2-vpaddd-3.c: New test.
* gcc.target/i386/avx2-vpaddw-3.c: Ditto.
* gcc.tar
>> I checked the attached patch, test results at
>> http://gcc.gnu.org/ml/gcc-testresults/2011-10/msg01377.html
>>
>> which are the same as with my suggested patch.
>>
>> Ok for the trunk?
>
> I probably don't have authority to approve this, but looks OK to me.
The libobjc bits are Ok for trunk.
Roman Zhuykov writes:
> [PATCH 4/9] Move the SMS pass earlier
> http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01811.html
I don't think this is a good idea. To get good results, SMS really
needs to run as close to the register allocator as possible, otherwise
later passes might disrupt the schedul
> gcc/
> * reload1.c (reload_regs_reach_end_p): Replace with...
> (reload_reg_rtx_reaches_end_p): ...this function.
> (new_spill_reg_store): Update commentary.
> (emit_input_reload_insns): Don't clear new_spill_reg_store here.
> (emit_output_reload_insns): Check reload
> I checked the attached patch, test results at
> http://gcc.gnu.org/ml/gcc-testresults/2011-10/msg01377.html
>
> which are the same as with my suggested patch.
>
> Ok for the trunk?
I probably don't have authority to approve this, but looks OK to me.
Paul
On 10/17/2011 03:39 PM, Gabriel Dos Reis wrote:
On Mon, Oct 17, 2011 at 8:32 AM, Paolo Carlini wrote:
Hi,
here submitter requests a more accurate error message for X.Y where X is a
pointer to class type. Thus the below, tested x86_64-linux.
Ok for mainline?
s/is of pointer type/has pointer t
Finally here is what I am checking in, which passes bootstrap with
--disable-checking --enable-languages=all,ada -- modulo this other bug
that breaks bootstrap as well
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50709.
It's been OKed off-line by Tom Tromey.
commit c1cd2be336ceec75cf40ac5f32cc4f72
On Mon, Oct 17, 2011 at 8:32 AM, Paolo Carlini wrote:
> Hi,
>
> here submitter requests a more accurate error message for X.Y where X is a
> pointer to class type. Thus the below, tested x86_64-linux.
>
> Ok for mainline?
s/is of pointer type/has pointer type/g
>
> Thanks,
> Paolo.
>
>
Sure,
Is simplier and also handles (A T[-IF] (B T-IF C) -> (A T B) T-IF C
case, which can happen by framing in conditions.
@@ -8380,13 +8400,65 @@ fold_truth_andor (location_t loc, enum t
lhs is another similar operation, try to merge its rhs with our
rhs. Then try to merge our lhs a
Hi,
here submitter requests a more accurate error message for X.Y where X is
a pointer to class type. Thus the below, tested x86_64-linux.
Ok for mainline?
Thanks,
Paolo.
//
/cp
2011-10-17 Paolo Carlini
PR c++/44524
* typeck.c (build_class_member_a
On 20/09/11 11:51, Andrew Stubbs wrote:
> On 09/09/11 12:55, Richard Earnshaw wrote:
>> The part number field is meaningless outside of the context of a a
>> specific vendor -- only taken as a pair can they refer to a specific
>> part. So why is the vendor field hard-coded rather than factored int
> "Tristan" == Tristan Gingold writes:
Tom> Another way to look at it is that there have been many changes to GCC's
Tom> DWARF output in the last few years. Surely these have broken these
Tom> DWARF consumers more than this change possibly could.
Tristan> Yes, but there is -gstrict-dwarf to
On Fri, Oct 14, 2011 at 20:27, Gabriel Charette wrote:
> Yes, I understand that.
>
> But when the second 2.pph is skipped when reading foo.pph, the reading
> of its line_table is also skipped (as foo.pph doesn't contain the
> line_table information for 2.h, 2.pph does and adds it when its
> includ
On Mon, Oct 17, 2011 at 2:22 PM, Kai Tietz wrote:
> Ok, I see. This might be profitable to do that. So fold_truth_op
> hunk looks like this
>
> @@ -5149,13 +5176,6 @@ fold_truthop (location_t loc, enum tree_
> build2 (BIT_IOR_EXPR, TREE_TYPE (ll_arg),
>
On 14/10/11 14:31, Bernd Schmidt wrote:
> On 07/13/11 16:03, Richard Earnshaw wrote:
>>> * config/arm/arm.c (store_multiple_sequence): Avoid cases where
>>> the base reg is stored iff compiling for Thumb1.
>>>
>>> * gcc.target/arm/pr49641.c: New test.
>
> Ping. Richard, you replied to
This patch adds a call to _commit() on _WIN32 for the FLUSH subroutine
and the FLUSH statement. It removes the _commit from gfortran's buf_flush.
Background:
* gfortran internally buffers the I/O, but it calls the nonbuffering
open/write/read functions (and not, e.g., fopen/fwrite/fread). On
U
Thanks!
K
On Sat, Oct 15, 2011 at 3:08 PM, Uros Bizjak wrote:
> On Sat, Oct 15, 2011 at 10:32 AM, Uros Bizjak wrote:
>
--- /dev/null
+++ b/gcc/testsuite/gcc.target/i386/fma_double_1.c
@@ -0,0 +1,19 @@
+/* { dg-do compile } */
+/* { dg-prune-output ".*warning: 'sseregpar
On Sun, Oct 16, 2011 at 22:09, Lawrence Crowl wrote:
> Factor the vector and chain output routines to remove boolean control
> parameters. The functions pph_out_tree_vec_1 and pph_out_chain_1 split
> their conditional parts of their implementation into their use cases,
> calling each other as need
Ok, I see. This might be profitable to do that. So fold_truth_op
hunk looks like this
@@ -5149,13 +5176,6 @@ fold_truthop (location_t loc, enum tree_
build2 (BIT_IOR_EXPR, TREE_TYPE (ll_arg),
ll_arg, rl_arg),
This fixes PR50729, so much for not implementing it by using
value-ranges ... opens up the door for more bugs ;)
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2011-10-17 Richard Guenther
PR tree-optimization/50729
* tree-vrp.c (extract_range
This changes alignment computation of a MEM during expansion
from MAX (TYPE_ALIGN (TREE_TYPE (exp)), get_object_alignment (exp))
to something weaker (with possibly less alignment) when we
were able to compute an explicit misaligned value (thus,
get_object_alignment_1 (exp, &misalign) would return
On Tue, Oct 11, 2011 at 04:19:52PM -0300, Alexandre Oliva wrote:
> Here's what I've got so far. Regstrapped on x86_64-linux-gnu and
> i686-linux-gnu. Ok to install?
I see
+FAIL: gcc.c-torture/compile/pr19080.c -O3 -g (internal compiler error)
+FAIL: gcc.c-torture/compile/pr19080.c -O3 -g (te
On Mon, Oct 17, 2011 at 6:48 AM, Paolo Carlini wrote:
> On 10/17/2011 01:44 PM, Gabriel Dos Reis wrote:
>>
>> On Mon, Oct 17, 2011 at 6:26 AM, Paolo Carlini
>> wrote:
>>>
>>> I would **strongly** oppose any change to 72 not strongly
>>> motivated at least by a comparison with other high quality f
Richard Guenther writes:
> This broke bootstrap on x86_64-linux.
>
> /space/rguenther/src/svn/trunk/libcpp/line-map.c: In function
> 'source_location linemap_macro_map_loc_to_exp_point(const line_map*,
> source_location)':
> /space/rguenther/src/svn/trunk/libcpp/line-map.c:628:12: error:
> variab
On 10/17/2011 01:44 PM, Gabriel Dos Reis wrote:
On Mon, Oct 17, 2011 at 6:26 AM, Paolo Carlini wrote:
I would **strongly** oppose any change to 72 not strongly
motivated at least by a comparison with other high quality front-ends
I love it when you make arguments like this. It makes me smile,
On Mon, Oct 17, 2011 at 6:26 AM, Paolo Carlini wrote:
> I would **strongly** oppose any change to 72 not strongly
> motivated at least by a comparison with other high quality front-ends
I love it when you make arguments like this. It makes me smile, and
I like smiling when I just get off bed :-)
On Mon, Oct 17, 2011 at 1:31 PM, Kai Tietz wrote:
> 2011/10/17 Richard Guenther :
>> On Mon, Oct 17, 2011 at 12:59 PM, Kai Tietz wrote:
>>> 2011/10/17 Richard Guenther :
On Fri, Oct 14, 2011 at 9:43 PM, Kai Tietz wrote:
> Hello,
>
> So I committed the gimplify patch separate. A
On Mon, Oct 17, 2011 at 6:29 AM, Richard Guenther wrote:
> On Mon, 17 Oct 2011, Gabriel Dos Reis wrote:
>
>> On Mon, Oct 17, 2011 at 6:08 AM, Richard Guenther wrote:
>>
>> > The initial patch, split between rev. 31343 and 31999 indeed added
>>
>> Thanks for helping tracking history.
>>
>> > + /*
2011/10/17 Richard Guenther :
> On Mon, Oct 17, 2011 at 12:59 PM, Kai Tietz wrote:
>> 2011/10/17 Richard Guenther :
>>> On Fri, Oct 14, 2011 at 9:43 PM, Kai Tietz wrote:
Hello,
So I committed the gimplify patch separate. And here is the remaining
fold-const patch.
The im
On Mon, Oct 17, 2011 at 6:19 AM, Paolo Carlini wrote:
> On 10/17/2011 01:16 PM, Gabriel Dos Reis wrote:
>>
>> On Mon, Oct 17, 2011 at 6:09 AM, Paolo Carlini
>> wrote:
>>>
>>> On 10/17/2011 12:56 PM, Gabriel Dos Reis wrote:
>
> Thus clearly the documentation is wrong ;)
;-)
On Mon, 17 Oct 2011, Gabriel Dos Reis wrote:
> On Mon, Oct 17, 2011 at 6:08 AM, Richard Guenther wrote:
>
> > The initial patch, split between rev. 31343 and 31999 indeed added
>
> Thanks for helping tracking history.
>
> > + /* Enable automatic line wrapping by default */
> > + set_message_
On 10/17/2011 01:24 PM, Gabriel Dos Reis wrote:
Again this argument is making a sort of revisionism. The 72 default
was added to g++, and other front-ends (in reality at the time, only C
could be affected) decided not to. Over the years, we have moved to
share more and more codes with other fro
On Mon, Oct 17, 2011 at 6:14 AM, Paolo Carlini wrote:
> On 10/17/2011 01:08 PM, Richard Guenther wrote:
>>
>> The initial patch, split between rev. 31343 and 31999 indeed added
>>
>> + /* Enable automatic line wrapping by default */
>> + set_message_length (72);
>>
>> to C++ lang_decode_option.
On 10/17/2011 01:16 PM, Gabriel Dos Reis wrote:
On Mon, Oct 17, 2011 at 6:09 AM, Paolo Carlini wrote:
On 10/17/2011 12:56 PM, Gabriel Dos Reis wrote:
Thus clearly the documentation is wrong ;)
;-)
Not necessarily. Paolo does not say why that line was added.
I don't remember adding that line
On Mon, Oct 17, 2011 at 12:59 PM, Kai Tietz wrote:
> 2011/10/17 Richard Guenther :
>> On Fri, Oct 14, 2011 at 9:43 PM, Kai Tietz wrote:
>>> Hello,
>>>
>>> So I committed the gimplify patch separate. And here is the remaining
>>> fold-const patch.
>>> The important tests here are in gcc.dg/tree-s
On 10/17/2011 01:08 PM, Richard Guenther wrote:
The initial patch, split between rev. 31343 and 31999 indeed added
+ /* Enable automatic line wrapping by default */
+ set_message_length (72);
to C++ lang_decode_option. Later it got appearantly lost somehow,
probably during some of the Great
On Mon, Oct 17, 2011 at 6:09 AM, Paolo Carlini wrote:
> On 10/17/2011 12:56 PM, Gabriel Dos Reis wrote:
>>>
>>> Thus clearly the documentation is wrong ;)
>>
>> ;-)
>> Not necessarily. Paolo does not say why that line was added.
>> I don't remember adding that line to change the default.
>
> Inde
On Mon, Oct 17, 2011 at 6:08 AM, Richard Guenther wrote:
> The initial patch, split between rev. 31343 and 31999 indeed added
Thanks for helping tracking history.
> + /* Enable automatic line wrapping by default */
> + set_message_length (72);
>
> to C++ lang_decode_option. Later it got appe
On 10/17/2011 12:56 PM, Gabriel Dos Reis wrote:
Thus clearly the documentation is wrong ;)
;-)
Not necessarily. Paolo does not say why that line was added.
I don't remember adding that line to change the default.
Indeed, as far as I can see, you added that line while *preserving* the
existing
On Mon, Oct 17, 2011 at 5:38 AM, Paolo Carlini wrote:
> On 10/17/2011 12:26 PM, Gabriel Dos Reis wrote:
>>
>> On Mon, Oct 17, 2011 at 4:42 AM, Paolo Carlini
>> wrote:
>>>
>>> FWIW, I still believe that tweaking the documentation to match the long
>>> standing reality, would be a small improvement
On Mon, 17 Oct 2011, Gabriel Dos Reis wrote:
> On Mon, Oct 17, 2011 at 5:53 AM, Richard Guenther wrote:
> > On Mon, 17 Oct 2011, Paolo Carlini wrote:
> >
> >> On 10/17/2011 12:26 PM, Gabriel Dos Reis wrote:
> >> > On Mon, Oct 17, 2011 at 4:42 AM, Paolo Carlini
> >> > wrote:
> >> > > FWIW, I still
Ping.
Could please someone check if my approach is OK and it is worth to
continue work on patch for PR50038?
Thanks
Ilya
2011/10/11 Ilya Enkovich :
> 2011/10/4 Richard Henderson :
>> On 10/04/2011 08:42 AM, Joseph S. Myers wrote:
>>> On Tue, 4 Oct 2011, Ilya Tocar wrote:
>>>
Hi everyone,
>>
1 - 100 of 131 matches
Mail list logo
