> PR rtl-optimization/48826
> * emit-rtl.c (try_split): When splitting a call that is followed
> by a NOTE_INSN_CALL_ARG_LOCATION, move the note after the new call.
OK if you move up the comment and merge it in the comment of the block. And it
would be nice to add the missing b
Hi!
- http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00895.html
P1 PR48973 4.6/4.7 fix for expansion of comparisons with signed
1 bit precision type
- http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00403.html
debug info improvement for unused parameters passed in memory
- http://gcc.gnu.org/ml
Not as many as Jakub, but still. :-)
Fix annoying gcov filename handling (gcov.c, 2 lines):
http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01380.html
Introduce -Wstack-usage:
http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01992.html
Extend TYPE_DECL_IS_STUB trick (dwarf2out.c, 1 line):
http:/
On 23 May 2011 03:07, Jason Merrill wrote:
> On 05/22/2011 05:11 PM, Jonathan Wakely wrote:
>>
>> As I said in the audit trail, -Winit-self is broken for C++ (PR
>> c++/34772)
>
> Not anymore. :)
Aha!
>> Even if -Winit-self wasn't broken I don't think it applies here,
>> there's no valid reason
Hello,
Here is a two lines patch, allowing to use debug_dominance_info and
debug_dominance_tree functions outside of gcc/dominance.c. For the
moment, those functions are not declared in any gcc/*.h files (as far as
I know after trying a grep). I have added them as external functions
into gcc/basic
On Mon, May 23, 2011 at 10:33:55AM +0200, Piervit wrote:
>
> Here is a two lines patch, allowing to use debug_dominance_info and
> debug_dominance_tree functions outside of gcc/dominance.c.
> +extern void debug_dominance_info (enum cdi_direction dir);
> +extern void debug_dominance_tree (enum cd
On Sun, May 22, 2011 at 10:51:30PM +0200, Eric Botcazou wrote:
> var-tracking doesn't fully work with dynamic stack realignment on x86 when a
> DRAP register is used for a function: the value of the incoming parameters is
> very quickly wrong (as soon as the DRAP register is clobbered), in partic
On Mon, May 23, 2011 at 1:14 AM, Xinliang David Li wrote:
> Hi, the following trial patch fixed PR 48988 which is a regression.
>
> Bootstrap and tested on x86/linux. Verified the reported failure is fixed.
>
> Ok for trunk?
Ok.
Thanks,
Richard.
> David
>
>
> 2011-05-22 David Li
>
> P
On Mon, 23 May 2011, Jakub Jelinek wrote:
> Hi!
>
> - http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00895.html
> P1 PR48973 4.6/4.7 fix for expansion of comparisons with signed
> 1 bit precision type
Ok.
I'll leave the rest to others.
Thanks,
Richard.
On Mon, May 23, 2011 at 9:59 AM, Eric Botcazou wrote:
> Not as many as Jakub, but still. :-)
>
>
> Fix annoying gcov filename handling (gcov.c, 2 lines):
> http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01380.html
Ok.
> Introduce -Wstack-usage:
> http://gcc.gnu.org/ml/gcc-patches/2011-03/msg0199
On Mon, May 23, 2011 at 10:33 AM, Piervit wrote:
> Hello,
>
> Here is a two lines patch, allowing to use debug_dominance_info and
> debug_dominance_tree functions outside of gcc/dominance.c. For the
> moment, those functions are not declared in any gcc/*.h files (as far as
> I know after trying a
> Do you mean something like the attached one? We don't have guality
> testsuite for ada and I think ada testcases are quite unreadable for most
> people. Definitely to me. Plus it is always better to have a single
> file testcase over 5 files.
Sure. Last time I mentioned the guality testsuite
Ping: C6X scheduler changes.
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00747.html
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00749.html
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00750.html
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00753.html
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00909.html
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00757.html
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00760.html
Bernd
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00992.html
Bernd
Le Mon, 23 May 2011 11:30:34 +0200,
Richard Guenther a écrit :
> On Mon, May 23, 2011 at 10:33 AM, Piervit wrote:
> > Hello,
> >
> > Here is a two lines patch, allowing to use debug_dominance_info and
> > debug_dominance_tree functions outside of gcc/dominance.c. For the
> > moment, those functi
As Arnaud suggested with one of my --enable-threads cleanup patches,
--enable-threads=gnat is unused and can go. The following patch
implements this.
Bootstrapped without regressions on i386-pc-solaris2.11.
While I didn't test the other affected configurations, I hope the
changes are sufficientl
> As Arnaud suggested with one of my --enable-threads cleanup patches,
> --enable-threads=gnat is unused and can go. The following patch
> implements this.
>
> Bootstrapped without regressions on i386-pc-solaris2.11.
>
> While I didn't test the other affected configurations, I hope the
> changes
As described in the PR, the go libgo test fails on Solaris since /bin is
a symlink to /usr/bin. Since there is other OS-specific code in
os_test.go, I'm using the follwing patch to fix this.
With that patch, libgo test results on i386-pc-solaris2.11 are clean.
Rainer
2011-05-20 Rainer
> > Fix annoying gcov filename handling (gcov.c, 2 lines):
> > http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01380.html
>
> Ok.
Thanks!
> > Introduce -Wstack-usage:
> > http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01992.html
>
> I wonder if we can't simply use -Wframe-larger-than but with
> the
On Mon, May 23, 2011 at 11:33:42AM +0200, Eric Botcazou wrote:
> > Do you mean something like the attached one? We don't have guality
> > testsuite for ada and I think ada testcases are quite unreadable for most
> > people. Definitely to me. Plus it is always better to have a single
> > file tes
--- Begin Message ---
On 14/05/11 15:43, Tobias Burnus wrote:
> Joern Rennecke wrote:
>> This patch hasn't been reviewed for a week:
>> http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00582.html
>>
>
> The Fortran bits are OK.
As is the Java bit.
Andrew.
--- End Message ---
On May 23, 2011, at 12:06 PM, Rainer Orth wrote:
> As Arnaud suggested with one of my --enable-threads cleanup patches,
> --enable-threads=gnat is unused and can go. The following patch
> implements this.
>
> Bootstrapped without regressions on i386-pc-solaris2.11.
>
> While I didn't test the
Arnaud Charlet writes:
> Certainly looks good to me, although I probably can't officially approve it.
I think you can since this only affects Ada. I'll wait a day or two if
anyone objects, then install the patch.
Thanks.
Rainer
--
-
On Mon, May 23, 2011 at 12:15 PM, Eric Botcazou wrote:
>> > Fix annoying gcov filename handling (gcov.c, 2 lines):
>> > http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01380.html
>>
>> Ok.
>
> Thanks!
>
>> > Introduce -Wstack-usage:
>> > http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01992.html
>>
>>
On 05/23/2011 12:33 PM, Rainer Orth wrote:
> Certainly looks good to me, although I probably can't officially approve it.
I think you can since this only affects Ada. I'll wait a day or two if
anyone objects, then install the patch.
It's okay for me too.
Paolo
While working to improve Solaris/x86 TLS support, Uros provided me with
a testcase to excercise all 4 TLS access models. Since there were
different issues affecting the different models, I've split the testcase
into one per access model so one can easier see what's broken.
Tested with the appropr
On Thu, May 19, 2011 at 4:04 PM, William J. Schmidt
wrote:
> As Richard Guenther suggested, I'm submitting a separate patch to add
> powi expansion to multiplies during the cse_sincos pass. This was
> originally submitted as part of a larger patch to fix PR46728.
>
> Richard noted that tree_expan
Paolo Bonzini writes:
> On 05/23/2011 12:33 PM, Rainer Orth wrote:
>>> > Certainly looks good to me, although I probably can't officially approve
>>> > it.
>> I think you can since this only affects Ada. I'll wait a day or two if
>> anyone objects, then install the patch.
>
> It's okay for me
We're confused when void * pointers enter memcpy folding. The
following fixes it.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2011-05-23 Richard Guenther
PR middle-end/15419
* builtins.c (fold_builtin_memory_op): Be less restrictive about
This fixes PR49115 - we cannot assume that a stmt that can throw
internally is a kill.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk,
queued for backporting.
Richard.
2011-05-23 Richard Guenther
PR tree-optimization/49115
* tree-ssa-alias.c (stmt_kill
DJ Delorie writes:
>> [build] More --enable-threads cleanup
>> http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00059.html
>>
>> This might well be obvious.
>
> Also, we usually leave those up to the target maintainers, since
> they're target specific. But if it's just a correlation be
On Thu, May 12, 2011 at 7:36 AM, Jakub Jelinek wrote:
> Hi!
>
> The read from a 1-bit signed bitfield initialized by a comparison
> is optimized into the comparison, which has that 1-bit signed bitfield
> comparison. Unfortunately that is still expanded as setting the result
> to 0 resp. 1 instea
On Mon, May 23, 2011 at 06:06:55AM -0700, H.J. Lu wrote:
> On Thu, May 12, 2011 at 7:36 AM, Jakub Jelinek wrote:
> > The read from a 1-bit signed bitfield initialized by a comparison
> > is optimized into the comparison, which has that 1-bit signed bitfield
> > comparison. Unfortunately that is s
On Sat, May 21, 2011 at 5:24 AM, Zdenek Dvorak wrote:
> Hi,
>
>> > Would it be possible to add the handling of these cases
>> > to estimate_numbers_of_iterations, rather than doing it just for ivopts?
>>
>> Yes, I did that now.
>>
>> Thanks,
>> - Tom
>>
>> 2011-05-05 Tom de Vries
>>
>> PR
On Mon, May 23, 2011 at 6:21 AM, Jakub Jelinek wrote:
> On Mon, May 23, 2011 at 06:06:55AM -0700, H.J. Lu wrote:
>> On Thu, May 12, 2011 at 7:36 AM, Jakub Jelinek wrote:
>> > The read from a 1-bit signed bitfield initialized by a comparison
>> > is optimized into the comparison, which has that 1-
Various places in the compiler grab TYPE_ARG_TYPES and grovel through it
when what they're really trying to do is index into the list of argument
types. The patch below introduces nth_arg_type for such situatiosn and
changes a hodgepodge of places to use it. You could, of course, use
function_arg
On Mon, May 23, 2011 at 3:53 PM, Nathan Froyd wrote:
> Various places in the compiler grab TYPE_ARG_TYPES and grovel through it
> when what they're really trying to do is index into the list of argument
> types. The patch below introduces nth_arg_type for such situatiosn and
> changes a hodgepodg
On 05/23/2011 10:05 AM, Richard Guenther wrote:
> On Mon, May 23, 2011 at 3:53 PM, Nathan Froyd
> wrote:
>> +/* Return the Nth argument type from FNTYPE. */
>> +
>> +tree
>> +nth_arg_type (const_tree fntype, int n)
>> +{
>> + function_args_iterator iter;
>> + tree t;
>> + int i;
>> +
>> + gc
On 05/17/2011 11:31 AM, Nathan Froyd wrote:
> On 05/10/2011 04:18 PM, Nathan Froyd wrote:
>> On 03/10/2011 11:23 PM, Nathan Froyd wrote:
>>> After all that, we can finally make tree_exp inherit from typed_tree.
>>> Quite anticlimatic.
>>
>> Ping. http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00559.
On 05/23/2011 04:20 AM, Jonathan Wakely wrote:
I still think this warning shouldn't be tied to -Winit-self. That
isn't included in -Wall or -Wextra, and it implies we're saying that a
self-referencing mem-initializer is a GNU extension to silence a
warning, when I think it's always a mistake and
On 05/23/2011 09:53 AM, Nathan Froyd wrote:
#define DECL_CONST_MEMFUNC_P(NODE) \
(DECL_NONSTATIC_MEMBER_FUNCTION_P (NODE) \
-&& CP_TYPE_CONST_P (TREE_TYPE (TREE_VALUE \
-
Committed.
2011-05-23 Richard Guenther
PR tree-optimization/40874
* g++.dg/tree-ssa/pr40874.C: New testcase.
Index: gcc/testsuite/g++.dg/tree-ssa/pr40874.C
===
--- gcc/testsuite/g++.dg/tree-ssa/pr40874.C (rev
While looking at c++/47263 I noticed that eh_ptr.cc doesn't compile in
C++0x mode because __safe_bool isn't defined, but is used. Is this the
way you'd like to fix that?
commit e018fa18d85e6974b3d66a12c5b7ae9f77b76683
Author: Jason Merrill
Date: Sun May 22 16:31:20 2011 -0400
* libsupc+
C++0x scoped enums don't implicitly convert to an integral type, so we
need to be careful about throwing away explicit conversions. In this
case, the static_cast(e) creates a NOP_EXPR which get_narrower
throws away, but 'e' itself is not a valid argument for &, so we need to
replace the NOP_EX
Committed.
Richard.
2011-05-23 Richard Guenther
* gcc.dg/tree-ssa/forwprop-11.c: Adjust and un-XFAIL.
Index: gcc/testsuite/gcc.dg/tree-ssa/forwprop-11.c
===
--- gcc/testsuite/gcc.dg/tree-ssa/forwprop-11.c (revision 1740
Just a missing case.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 047661c30a4c6174fc8b994fcdd3eb344d10609c
Author: Jason Merrill
Date: Sun May 22 15:06:19 2011 -0400
PR c++/48617
* pt.c (invalid_nontype_parm_type_p): Allow DECLTYPE_TYPE.
diff --git a/gcc/cp/pt.c b/gcc/cp/p
A constructor defaulted outside the class body is considered
user-provided, so we need to handle instantiating it. But functions
defaulted inside the class body we still want to treat the same as
implicitly declared functions, so the test checks for that as well.
Tested x86_64-pc-linux-gnu, a
On 05/18/11 06:33, Jakub Jelinek wrote:
> Hi!
>
> The attached testcase ICEs on arm, because extract_fixed_bit_field
> with tmode SImode (and SImode non-NULL target) decides to use DImode for
> the signed shifts, but doesn't clear target and thus attempts to use
> that SImode target for DImode shi
On Mon, May 23, 2011 at 4:18 PM, Nathan Froyd wrote:
> On 05/17/2011 11:31 AM, Nathan Froyd wrote:
>> On 05/10/2011 04:18 PM, Nathan Froyd wrote:
>>> On 03/10/2011 11:23 PM, Nathan Froyd wrote:
After all that, we can finally make tree_exp inherit from typed_tree.
Quite anticlimatic.
>>>
In this case, we ran into an access control error trying to print the
meaning of a decltype while dumping instantiation context. The simplest
answer is to just disable access control during that process rather than
try to push into the right context.
Tested x86_64-pc-linux-gnu, applying to tr
On Mon, May 23, 2011 at 4:50 PM, Jason Merrill wrote:
> C++0x scoped enums don't implicitly convert to an integral type, so we need
> to be careful about throwing away explicit conversions. In this case, the
> static_cast(e) creates a NOP_EXPR which get_narrower throws away, but
> 'e' itself is n
On 05/23/2011 04:45 PM, Jason Merrill wrote:
While looking at c++/47263 I noticed that eh_ptr.cc doesn't compile in
C++0x mode because __safe_bool isn't defined, but is used. Is this
the way you'd like to fix that?
Jason, the change by itself seems indeed ok, but really I don't think we
should
Normally, we try to be somewhat permissive about overload resolution,
accepting near-matches when available in order to give better
diagnostics about why they aren't complete matches. But this doesn't
apply in templates, since we don't actually perform the conversions that
would generate the d
On 05/22/2011 02:24 PM, Tom de Vries wrote:
> Now that struct tree_type does not exist anymore, 'sizeof (struct tree_type)'
> generates an error in the following assert in fold_checksum_tree:
> ...
> gcc_assert ((sizeof (struct tree_exp) + 5 * sizeof (tree)
> <= sizeof (struct tree_f
This is the last (planned) bit for the regular type merging machinery
(there is still some issues with TYPE_CANONICAL that I will try to
address next). It avoids merging types which have different names
as that leads to interesting debuginfo issues.
LTO-bootstrapped and tested on x86_64-unknown-
Hi Alex, Hi Jeff, Hi Richard,
Consider the following small test case:
% cat test1.c
extern int g (void) __attribute__ ((visibility("protected")));
int f (void) { return g (); }
% cat test2.c
extern int g(void) __attribute__ ((visibility("protected")));
int i;
int g
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 05/13/11 11:13, Vladimir Makarov wrote:
> The patch fixes PR48633 described on
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48633.
>
> The patch was successfully bootstrapped on x86-64.
>
> Ok to commit?
>
> 2011-05-13 Vladimir Makarov
>
>
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 05/13/11 10:58, Vladimir Makarov wrote:
> The following patch solves the problem described on
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48971
>
> The patch was successfully bootstrapped on x86-64 and checked for ppc64,
> ia64, i686-linux, i686-
On Mon, May 23, 2011 at 8:18 AM, Richard Guenther wrote:
>
> This is the last (planned) bit for the regular type merging machinery
> (there is still some issues with TYPE_CANONICAL that I will try to
> address next). It avoids merging types which have different names
> as that leads to interestin
On 05/23/2011 11:01 AM, Richard Guenther wrote:
Ok. Though I wonder where you enter folding before you check semantic
validity.
The problem is in c-common:shorten_binary_op, which uses get_narrower
and then calls common_type on the types of the results. There is no
common type between a sco
On Mon, May 23, 2011 at 5:34 PM, H.J. Lu wrote:
> On Mon, May 23, 2011 at 8:18 AM, Richard Guenther wrote:
>>
>> This is the last (planned) bit for the regular type merging machinery
>> (there is still some issues with TYPE_CANONICAL that I will try to
>> address next). It avoids merging types w
On Mon, May 23, 2011 at 5:38 PM, Jason Merrill wrote:
> On 05/23/2011 11:01 AM, Richard Guenther wrote:
>>
>> Ok. Though I wonder where you enter folding before you check semantic
>> validity.
>
> The problem is in c-common:shorten_binary_op, which uses get_narrower and
> then calls common_type o
G++ was omitting code for the exception specification on the lambda
because it was confusing it with an synthesized constructor. This patch
fixes the test to match the comment better; not all artificial methods
are [cd]tors.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 79adecb0b52f58
This patch fixes an interoperability issue with code generated by ARM's EABI
compiler.
Unlike the generic C++ ABI, which always catches pointers by value, ARM's ABI
only catches pointers by value when there's the possibility of derived->base
conversion happening. The ARM __cxa_type_matcher ca
On 05/23/2011 04:52 PM, Nathan Sidwell wrote:
> This patch fixes an interoperability issue with code generated by ARM's EABI
> compiler.
> This patch results has been tested for arm-linux, and independently tested by
> ARM with mixed RVCT-generated code confirming the defect has been fixed.
>
>
Hi,
This patch fixes ICE by saving the initial options after checking
vzeroupper. OK for trunk?
Thanks.
H.J.
---
gcc/
2011-05-23 H.J. Lu
PR target/47315
* config/i386/i386.c (ix86_option_override_internal): Save the
initial options after checking vzeroupper.
gcc/t
Maybe we can offer an additional compression format like lzma .xz
which brings down 4.5.3 compressed size from 66MB bz2 to 52MB (with
-7).
But that can be decided separately.
Yes ... on the other hand, this does not address the other issue,
which is the
amount of disk space that you need
On Mon, May 23, 2011 at 5:59 PM, H.J. Lu wrote:
> This patch fixes ICE by saving the initial options after checking
> vzeroupper. OK for trunk?
>
> Thanks.
>
>
> H.J.
> ---
> gcc/
>
> 2011-05-23 H.J. Lu
>
> PR target/47315
> * config/i386/i386.c (ix86_option_override_internal):
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 05/23/11 09:21, Nick Clifton wrote:
> Hi Alex, Hi Jeff, Hi Richard,
>
> Consider the following small test case:
>
> % cat test1.c
> extern int g (void) __attribute__ ((visibility("protected")));
> int f (void) { return g (); }
>
>
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 05/12/11 08:36, Jakub Jelinek wrote:
> Hi!
>
> The read from a 1-bit signed bitfield initialized by a comparison
> is optimized into the comparison, which has that 1-bit signed bitfield
> comparison. Unfortunately that is still expanded as setting
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 05/19/11 07:37, Bernd Schmidt wrote:
> In the scheduler initialization code, we want to set up the h_i_d array.
> So we call haifa_init_h_i_d, which has a table of function pointers
> which it passes to sched_scan, which copies the function pointer
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 05/23/11 01:48, Jakub Jelinek wrote:
>
> - http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00403.html
> debug info improvement for unused parameters passed in memory
OK.
>
> - http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01454.html
> PR49032 f
On Mon, May 23, 2011 at 11:33:42AM +0200, Eric Botcazou wrote:
> > Anyway, IMHO the desired outcome is that the
> > parameters will have DW_OP_fbreg {0,4} as their location, so perhaps the
> > DRAP register needs to be remapped somehow during adjust_insn.
>
> So you don't care about their locatio
Hi!
This patch shuts up a warning about VLA bounds expression being unused.
start_decl adds the (usually SAVE_EXPR) expression to statements to make
sure it is evaluated, but if that happens in statement expression where
unused value warnings are postponed until we know what is the return value,
i
Hi!
This is something I didn't notice in my testing because my ld doesn't
support plugins and thus -flto didn't act as -fwhole-file.
The bug got fixed on the trunk by changing build_int_cst, which is
too risky to backport, so this patch instead just
adjusts the single place relevant to this patch
On Mon, 23 May 2011, Jakub Jelinek wrote:
> Hi!
>
> This patch shuts up a warning about VLA bounds expression being unused.
> start_decl adds the (usually SAVE_EXPR) expression to statements to make
> sure it is evaluated, but if that happens in statement expression where
> unused value warnings
> there's nothing that could affect them.
Famous last words :-)
As described in
[testsuite] Provide TLS access model testcases
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01601.html
both the 32 and 64-bit TLS GD and LD execution tests fail on Solaris 2
with Sun ld unless you have a very recent Solaris 11 version (snv_164).
This happens becau
Eric Botcazou writes:
>> PR rtl-optimization/48826
>> * emit-rtl.c (try_split): When splitting a call that is followed
>> by a NOTE_INSN_CALL_ARG_LOCATION, move the note after the new call.
>
> OK if you move up the comment and merge it in the comment of the
> block. And it would b
Richard, thanks for the excellent comments. I really appreciate your
help. I've implemented them all, and bootstrap/regtest succeeds on
powerpc target. Here's the revised patch for your consideration.
Thanks,
Bill
2011-05-23 Bill Schmidt
* tree-ssa-math-opts.c (powi_table): New.
Hi,
We use processor model and family to detect processor. But some
processors may have some ISAs disabled. This patch always adds
-mno-XXX. OK for trunk and 4.6 if there are no regressions?
Thanks.
H.J.
---
2011-05-23 H.J. Lu
PR target/49128
* config/i386/driver-i386.c (h
On Mon, 23 May 2011 11:56:32 +0200
Piervit wrote:
> Le Mon, 23 May 2011 11:30:34 +0200,
> Richard Guenther a écrit :
>
> > On Mon, May 23, 2011 at 10:33 AM, Piervit wrote:
> > > Hello,
> > >
> > > Here is a two lines patch, allowing to use debug_dominance_info and
> > > debug_dominance_tree fu
On Mon, May 23, 2011 at 11:10:26AM -0700, H.J. Lu wrote:
> + const char *sse4_2 = has_sse4_2 ? " -msse4.2" : " -msse4.2";
> + const char *sse4_1 = has_sse4_1 ? " -msse4.1" : " -msse4.1";
Missing "no-" here on both lines.
Jakub
On Mon, May 23, 2011 at 11:16 AM, Jakub Jelinek wrote:
> On Mon, May 23, 2011 at 11:10:26AM -0700, H.J. Lu wrote:
>> + const char *sse4_2 = has_sse4_2 ? " -msse4.2" : " -msse4.2";
>> + const char *sse4_1 = has_sse4_1 ? " -msse4.1" : " -msse4.1";
>
> Missing "no-" here on both lines.
>
H
On Mon, May 23, 2011 at 7:22 PM, Rainer Orth
wrote:
> As described in
>
> [testsuite] Provide TLS access model testcases
> http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01601.html
>
> both the 32 and 64-bit TLS GD and LD execution tests fail on Solaris 2
> with Sun ld unless you have
Hi,
this patch fixes an obvious bootstrap issue caused by trying to assign
a constant pointer to an none-constant.
Index: adaint.c
===
--- adaint.c(revision 174060)
+++ adaint.c(working copy)
@@ -3367,8 +3367,8 @@
char *
__
This patch splits the generated file options.c into two parts, one
shared with the driver and one not, so avoiding the need to build a
special gcc-options.o by defining GCC_DRIVER when compiling options.c.
The GCC_DRIVER macro, being obsolete, is removed. It ought to be
possible to simplify the c
> It sounded like you were more concerned with having the comments staying
> in sync than the actual specifics, so I went ahead and installed the
> patch below.
That's fine, thanks.
--
Eric Botcazou
On Mon, May 23, 2011 at 6:56 PM, Jakub Jelinek wrote:
> Hi!
>
> This is something I didn't notice in my testing because my ld doesn't
> support plugins and thus -flto didn't act as -fwhole-file.
> The bug got fixed on the trunk by changing build_int_cst, which is
> too risky to backport, so this p
On Mon, May 23, 2011 at 8:12 PM, Basile Starynkevitch
wrote:
> On Mon, 23 May 2011 11:56:32 +0200
> Piervit wrote:
>
>> Le Mon, 23 May 2011 11:30:34 +0200,
>> Richard Guenther a écrit :
>>
>> > On Mon, May 23, 2011 at 10:33 AM, Piervit wrote:
>> > > Hello,
>> > >
>> > > Here is a two lines patc
On 05/23/2011 04:23 PM, Richard Guenther wrote:
>> So I don't buy Richie's argument. Otherwise, someone would propose a
>> patch to remove the hundreds of debug_ declarations in public header
>> files (i.e. those visible to plugins), and if he did, I hope such a
>> naughty patch won't be accepted.
On Mon, May 23, 2011 at 10:42 PM, Nathan Froyd wrote:
> On 05/23/2011 04:23 PM, Richard Guenther wrote:
>>> So I don't buy Richie's argument. Otherwise, someone would propose a
>>> patch to remove the hundreds of debug_ declarations in public header
>>> files (i.e. those visible to plugins), and i
Hi,
This test case gcc.dg/tree-ssa/ivopt_infer_2.c started failing due to
http://gcc.gnu.org/ml/gcc-cvs/2011-05/msg00837.html.
test case:
...
#ifndef TYPE
#define TYPE char*
#endif
extern int a[];
/* Can not infer loop iteration from array -- exit test can not be replaced.
*/
void foo (int i_wi
Add the initializtion for eof_pos. eof_pos may not be initialized properly
when part of gcda files got merged.
Tested with gcc bootstrap and regression tests.
2011-05-23 Rong Xu
* gcc/libgcov.c (gcov_merge_gcda_file): initialize eof_pos for
every invocation.
Index: gcc/libgc
ok for google/main.
David
On Mon, May 23, 2011 at 2:06 PM, Rong Xu wrote:
> Add the initializtion for eof_pos. eof_pos may not be initialized properly
> when part of gcda files got merged.
>
> Tested with gcc bootstrap and regression tests.
>
> 2011-05-23 Rong Xu
>
> * gcc/libgcov.c (g
Richard,
While working on the next patch, I ran into a scenario that will apply
to this one as well. If the call statement that calls powi contains
vdef/vuse information, it is lost by this replacement. For example,
# .MEM_20 = VDEF <.MEM_19(D)>
D.1980_3 = __builtin_powf (D.1979_2, 2.0e=0);
is
I completed re-testing everything.
It turns out I cannot reproduce the original error on gcc-4.4 (rev 173968)
So, I am submitting only the patch that I tested for gcc-4.5/4.6/4.7
Regression tested for e500mc target on:
4.5: Revision: 173928
4.6: Revision: 173936
trunk: Revision: 173966
The patch
On 05/23/2011 11:38 AM, Jason Merrill wrote:
Do you think I should change shorten_binary_op instead?
Thus.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 683f3617b323a9d8f62fc5f002d48948eb3b457b
Author: Jason Merrill
Date: Mon May 23 11:57:40 2011 -0400
PR c++/48106
* c-co
The main problem in this PR was that build_const_cast_1 hadn't been
updated to support rvalue references, so the first patch does that. The
second patch fixes a couple of minor issues that I noticed while
investigating this; neither of them should affect observable semantics,
but they match th
1 - 100 of 108 matches
Mail list logo
