On 12-10-11 4:17 PM, Steven Bosscher wrote:
Hello,
IRA uses record_loop_exits() to cache the loop exit edges, but due to
a code ordering bug the edges are not actually recorded.
record_loop_exits() starts with:
if (!current_loops)
return;
I have no idea why record_loop_exits is here. L
On 12-10-11 12:56 PM, Peter Bergner wrote:
On Thu, 2012-10-11 at 11:35 -0400, Vladimir Makarov wrote:
+ /* Call insn might have not references for pseudos besides
+in CALL_INSN_FUNCTION_USAGE but we don't count them in
+insn_bitmap of corresponding lra_reg_info as
On 10/04/2012 11:50 AM, Richard Sandiford wrote:
Hi Vlad,
This message is for lra-assigns.c. Sorry for the piecemeal reviews,
never sure when I'll get time...
+/* This file contains a pass mostly assigning hard registers to reload
+ pseudos. There is no any RTL code transformation on this
The following patch implements most Richard's proposals for LRA
lra-spills.c and lra-coalesce.c files.
The patch was successfully bootstrapped on x86/x86-64.
Committed as rev. 192389.
2012-10-11 Vladimir Makarov
* target.def (register_priority): Change value interpretation.
*
On 12-10-11 5:53 PM, Peter Bergner wrote:
On Tue, 2012-10-02 at 10:57 -0400, Vladimir Makarov wrote:
Chaitin-Briggs literature does not discuss the termination, just saying
that live-ranges shortening will result to assigning hard regs to all
necessary pseudos which is not clearly guaranteed. Th
> -Original Message-
> From: Steven Bosscher [mailto:stevenb@gmail.com]
> Sent: Friday, October 12, 2012 7:04 AM
> To: Bin Cheng
> Cc: Jeff Law; gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH RFA] Implement register pressure directed hoist pass
>
> On Thu, Oct 11, 2012 at 8:50 AM, Bin
Thanks for the comments. As David wrote, the intent of the patch is
not to do a general purpose scheduling, but to compensate for the
possible live range lengthening introduced by reassociation.
On Thu, Oct 11, 2012 at 6:16 AM, Richard Biener
wrote:
> On Thu, Oct 11, 2012 at 3:52 AM, Easwaran Ra
Oleg Endo wrote:
> Anyway, regardless of this failure, the attached patch for SH should be
> applicable. OK?
OK.
Regards,
kaz
Oleg Endo wrote:
> This one further improves T bit stores and conditional branches on SH
> for cases like described in comment #53 in the PR.
> Tested on rev 192200 with
> make -k check RUNTESTFLAGS="--target_board=sh-sim
> \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}"
>
> and no
Hi,
testcase committed, closed as fixed in 4.8.0.
Thanks,
Paolo.
//
2012-10-11 Paolo Carlini
PR c++/51878
* g++.dg/cpp0x/decltype45.C: New.
Index: g++.dg/cpp0x/decltype45.C
===
--- g++.dg/cpp
The md.texi entry for nonlocal_goto_receiver says "A typical reason
why you might need this pattern is if some value, such as a pointer to
a global table, must be restored when the frame pointer is restored.
Note that a nonlocal goto only occurs within a unit-of-translation, so
a global table point
Hi Janus,
ping!
2012/10/7 Janus Weil :
Hi all,
here is a rather straightforward patch, which does 'recursive'
checking of dummy procedures.
Regtested on x86_64-unknown-linux-gnu. Ok for trunk?
This is OK. Thanks for the patch!
Thomas
Hi,
Here is the initial test results of gcc asan patch, and it shows us
some missing features in gcc but existing in llvm.
[1]. gcc regression test for gcc-asan passes.
[2]. llvm regression tests for gcc-asan: 18 failures in 123 for tests
written in google test and 24 failures in 28 for tests writ
On Thu, Oct 11, 2012 at 8:50 AM, Bin Cheng wrote:
> + /* x+y won't be hoisted without defaultly enabled "-fira-hoist-pressure",
defaulty comment.
> + kinds of code motion(including code hoisting) in a unified way.
needs space between "motion" and "(".
> + flow graph, given if it can rea
On Thu, 2012-10-11 at 11:35 -0400, Vladimir Makarov wrote:
> + /* Call insn might have not references for pseudos besides
> +in CALL_INSN_FUNCTION_USAGE but we don't count them in
> +insn_bitmap of corresponding lra_reg_info as they don't
> +need reloads.
On Thu, 2012-10-11 at 23:11 +0800, Chung-Lin Tang wrote:
> This patch set has been committed, thanks to all maintainers who
> reviewed the respective parts.
>
> Thanks,
> Chung-Lin
>
This broke the recently added thread pointer built-ins on SH, but I was
prepared for that, so no problem here. T
Hello,
this patch brings VEC_COND_EXPR closer to what the doc now says. The
non-comparison case and gimplification are dead paths currently, but it
seems convenient to introduce them now. Note that by using the generic
ternary gimplification code, it will produce VEC_COND_EXPR with an
SSA_NAM
Hi,
not necessary anymore thanks to the resolution of c++/51213. Tested
x86_64-linux multilib, committed.
Thanks,
Paolo.
///
2012-10-12 Paolo Carlini
* include/bits/hashtable_policy.h: Revert libstdc++/53067 quick
hacks thanks to the resolution of c++/5
Hello,
this is an old patch that Paolo dug out of bugzilla and where I only
changed the error message. It fixes an ICE on invalid code.
Note that the code uses error() like the nearby code so the caret appears
on the RHS while the message is about the LHS. Paolo offered to look into
it as a
On Fri, Oct 12, 2012 at 12:00 AM, Ian Lance Taylor wrote:
> On Thu, Oct 11, 2012 at 1:25 PM, Steven Bosscher wrote:
>> On Thu, Oct 11, 2012 at 3:16 PM, Richard Biener wrote:
>>> NB, the whole reassoc code needs a re-write to avoid the excessive
>>> stmt modifications when it does nothing. So I'd v
On Thu, Oct 11, 2012 at 1:25 PM, Steven Bosscher wrote:
> On Thu, Oct 11, 2012 at 3:16 PM, Richard Biener
> wrote:
>> NB, the whole reassoc code needs a re-write to avoid the excessive
>> stmt modifications when it does nothing. So I'd very much rather avoid
>> adding anything to reassoc until t
Hello,
This one further improves T bit stores and conditional branches on SH
for cases like described in comment #53 in the PR.
Tested on rev 192200 with
make -k check RUNTESTFLAGS="--target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}"
and no new failures.
Cheers,
On Tue, 2012-10-02 at 10:57 -0400, Vladimir Makarov wrote:
> Chaitin-Briggs literature does not discuss the termination, just saying
> that live-ranges shortening will result to assigning hard regs to all
> necessary pseudos which is not clearly guaranteed. There is the same
> problem in LRA. S
On Thu, Oct 11, 2012 at 11:44 PM, Teresa Johnson wrote:
> + mode = targetm.machine_mode_from_attr_mode(insn);
Nit: space between "..._mode" and "(".
A test case would also be Nice To Have.
Looks OK to me otherwise, but I can't approve it.
Ciao!
Steven
Hi all,
here is an OOP patch for the above PR, which has two disconnected parts:
1) It fixes a problem with ASSOCIATED, when it is fed a CLASS-valued
function as argument (i.e. the ICE in the bug title). This is the
trans-intrinsic part of the patch. Instead of adding the _data
component to the e
Revised patch to address conservative behavior in redundant extend
elimination that was resulting in redundant extends not being
removed. Now uses a new target hook machine_mode_from_attr_mode
which is currently enabled only for i386.
Bootstrapped and tested on x86_64-unknown-linux-gnu. Is this ok
ping!
2012/10/7 Janus Weil :
> Hi all,
>
> here is a rather straightforward patch, which does 'recursive'
> checking of dummy procedures.
>
> Regtested on x86_64-unknown-linux-gnu. Ok for trunk?
>
> Cheers,
> Janus
>
>
> 2012-10-06 Janus Weil
>
> PR fortran/40453
> * interface.
On 10/11/2012 01:56 PM, Aldy Hernandez wrote:
> PR middle-end/54893
> * trans-mem.c (diagnose_tm_1_op): Allow volatiles inside relaxed
> transactions.
Ok.
r~
Hi Tobias,
> gfortran supports OPTIONAL since quite some time - also some more
> complicated cases involving ELEMENTAL or the new F2008 addition, but as
> testing showed, the support is still incomplete; especially with polymorphic
> arguments there were several bugs.
>
> Besides a simple absent a
Apparently we were a bit too aggressive on disallowing volatiles inside
transactions. Torvald seems to agree that volatiles should be allowed
inside relaxed transactions, since they will go into serial irrevocable
mode anyhow.
The following patch fixes the PR.
I did not include a test that f
Hello!
2012-10-11 Uros Bizjak
* config/alpha/alpha.md (IMODE): New mode iterator.
(I124MODE): Ditto.
(I248MODE): Ditto.
(modesuffix): Handle QI and HI modes.
(zero_extendqi2): Macroize insn from zero_extendqi{hi,si,di}2
using I248MODE mode iterat
ok.
thanks,
David
On Thu, Oct 11, 2012 at 1:21 PM, Easwaran Raman wrote:
> This patch moves the two argument delete operator into its own file.
> When a program provides its own definition of operator delete (void
> *), but not operator delete (void *, size_t), we could end up linking
> two fil
On Thu, Oct 11, 2012 at 1:23 PM, Lawrence Crowl wrote:
> On 10/10/12, Xinliang David Li wrote:
>> In a different thread, I proposed the following alternative to 'try_xxx':
>>
>> template T* symbol::cast_to(symbol* p) {
>>if (p->is())
>> return static_cast(p);
>>return 0;
>> }
>>
>>
On Thu, Oct 11, 2012 at 3:16 PM, Richard Biener
wrote:
> NB, the whole reassoc code needs a re-write to avoid the excessive
> stmt modifications when it does nothing. So I'd very much rather avoid
> adding anything to reassoc until that rewrite happened.
IMHO it's fair to Easwaran to hold up a p
On 10/10/12, Xinliang David Li wrote:
> In a different thread, I proposed the following alternative to 'try_xxx':
>
> template T* symbol::cast_to(symbol* p) {
>if (p->is())
> return static_cast(p);
>return 0;
> }
>
> cast:
>
> template T& symbol:as(symbol* p) {
>assert(p->is())
This patch moves the two argument delete operator into its own file.
When a program provides its own definition of operator delete (void
*), but not operator delete (void *, size_t), we could end up linking
two files that define _ZdlPv resulting in a linker error. Bootstraps
with no test regression
Hello,
IRA uses record_loop_exits() to cache the loop exit edges, but due to
a code ordering bug the edges are not actually recorded.
record_loop_exits() starts with:
if (!current_loops)
return;
So ira.c should set current_loops before calling record_loop_exits.
With the current order, the
On Thu, Oct 11, 2012 at 08:11:48PM +0200, Dominique Dhumieres wrote:
> Jack,
>
> > > That's PR 54407, so not a regression.
> >
> > ... The pthreads support is definitely improved in darwin11 and later.
>
> Do you confirm that 30_threads/condition_variable/54185.cc is passing on
> darwin11 and 12
On Thu, Oct 11, 2012 at 11:12 AM, Jakub Jelinek wrote:
> On Thu, Oct 11, 2012 at 10:31:58AM -0700, Xinliang David Li wrote:
>> > +#define PROB_VERY_UNLIKELY (REG_BR_PROB_BASE / 2000 - 1)
>> > +#define PROB_ALWAYS(REG_BR_PROB_BASE)
>> > +
>>
>>
>> Does it belong here ? -- looks like
How about this patch?
2012/10/11 Rainer Orth :
> Pavel Chupin writes:
>
>> Could you try this fix?
>
> Duplicating this much info from libgcc is certainly the wrong (read:
> unmaintainable) approach.
>
> Rainer
>
> --
> -
On Thu, Oct 11, 2012 at 10:31:58AM -0700, Xinliang David Li wrote:
> > +#define PROB_VERY_UNLIKELY (REG_BR_PROB_BASE / 2000 - 1)
> > +#define PROB_ALWAYS(REG_BR_PROB_BASE)
> > +
>
>
> Does it belong here ? -- looks like they can be generally useful for others.
Perhaps, but then i
Jack,
> > That's PR 54407, so not a regression.
>
> ... The pthreads support is definitely improved in darwin11 and later.
Do you confirm that 30_threads/condition_variable/54185.cc is passing on
darwin11 and 12?
If yes, I'll test a patch to skip it on darwin10 and older.
Dominique
On Thu, Oct 11, 2012 at 06:46:47PM +0100, Jonathan Wakely wrote:
> On 11 October 2012 17:05, Jack Howarth wrote:
> >
> > Jonathan,
> > The committed patch results in the expected this_thread::yield() and
> > this_thread::sleep_for()
> > support. I did run into a regression at -m32/-m64 on darw
>>> here is a small patch for a wrong-code regression with polymorphic
>>> allocation. The problem is that we falsely detect the allocation
>>> variable to be a polymorphic array (although it is a scalar). For
>>> further details see the PR, in particular comment 4.
>>>
>>> Regtested on x86_64-unkn
> Tested on x86_64. OK for trunk?
Looks OK, with some very minor comments and questions below.
> static void
> -consume_until_semi (bool immediate)
> +consume_until_eos (void)
> {
> - if (immediate && token () != ';')
> -require (';');
>for (;;)
> switch (token ())
>{
>
Hi Diego,
>> /* Build
>> - (base_addr >> ASAN_SHADOW_SHIFT) | targetm.asan_shadow_offset ().
>> */
>> + (base_addr >> ASAN_SHADOW_SHIFT) + targetm.asan_shadow_offset ().
>> */
>
>
> Hm, I wonder if this is a documentation problem or we're generating bad
> runtime code. Wei, you tested
On 11 October 2012 17:05, Jack Howarth wrote:
>
> Jonathan,
> The committed patch results in the expected this_thread::yield() and
> this_thread::sleep_for()
> support. I did run into a regression at -m32/-m64 on darwin10 (but not
> darwin11/12).
>
> WARNING: program timed out.
> FAIL: 30_thr
On Thu, Oct 11, 2012 at 9:38 AM, Jakub Jelinek wrote:
> Hi!
>
> Building trees, then gimplifying it, is unnecessarily expensive.
> This patch changes build_check_stmt to emit GIMPLE directly, and
> a couple of small cleanups here and there. Also, I'm using a different
> alias set for the shadow m
On Thu, Oct 11, 2012 at 01:14:31PM -0400, Diego Novillo wrote:
> On 2012-10-11 12:38 , Jakub Jelinek wrote:
>
> >- gimple_seq seq, stmts;
> >- tree shadow_type = size_in_bytes == 16 ?
> >- short_integer_type_node : char_type_node;
> >- tree shadow_ptr_type = build_pointer_type (shadow_type
On Oct 4, 2012, Jakub Jelinek wrote:
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
This is ok, except for one nit.
> 2012-10-04 Jakub Jelinek
> PR debug/54796
> * rtl.h: Document jump flag on VALUE.
> * cselib.h (cselib_set_value_sp_based,
>
On 2012-10-11 12:38 , Jakub Jelinek wrote:
- gimple_seq seq, stmts;
- tree shadow_type = size_in_bytes == 16 ?
- short_integer_type_node : char_type_node;
- tree shadow_ptr_type = build_pointer_type (shadow_type);
- tree uintptr_type = lang_hooks.types.type_for_mode (ptr_mode,
-
On Thu, Oct 11, 2012 at 9:19 AM, Gopalasubramanian, Ganesh
wrote:
> The attached patch (Patch.txt) enables the next version of AMD's bulldozer
> core.
Please handle new sseshuf type attribute in various attribute
calculations. You should at least add it to unit attribute
calculation, but please
On Thu, Oct 11, 2012 at 1:44 AM, Rainer Orth
wrote:
> Ian Lance Taylor writes:
>
>> On Wed, Oct 10, 2012 at 7:13 AM, Rainer Orth
>> wrote:
>>>
>>> Solaris 10 Update 10 or sufficiently recent linker patches introduced
>>> dl_iterate_phdr on S10 as a backport from Solaris 11, but unlike S11, it
>>
Hi!
Building trees, then gimplifying it, is unnecessarily expensive.
This patch changes build_check_stmt to emit GIMPLE directly, and
a couple of small cleanups here and there. Also, I'm using a different
alias set for the shadow memory accesses, those obviously can't alias any
other accesses.
O
Hi,
while looking into RTL loop peeling micopmilation I found that we now do a lot
of
RTL loop peeling for loops of the form
int a[2];
test(int c)
{
int i;
for (i=0;isrc && stmt == last_stmt (exit->src))
+ if (exit && body[i] == exit->src && stmt == last_stmt (exit->src))
On Thu, Oct 11, 2012 at 12:18:59AM +0100, Jonathan Wakely wrote:
> On 11 October 2012 00:09, Jonathan Wakely wrote:
> > On 9 October 2012 18:49, Benjamin De Kosnik wrote:
> >>
> >>> I don't like the sched_yield macro being set there because it's
> >>> detected correctly by configure anyway, but I'm
On 2012-10-11 10:32 , Jakub Jelinek wrote:
2012-10-11 Jakub Jelinek
* toplev.c (process_options): Warn and turn off -fasan
if not supported by target.
* asan.c: Include target.h.
(asan_scale, asan_offset_log_32, asan_offset_log_64,
asan_offset_log): Re
The following patch fixes scheduler crash on a SPEC2006 test because LRA
did not process CALL_INSN_FUNCTION_USAGE.
The patch was successfully bootstrapped and tested on x86/x86-64.
Committed as rev. 192362.
2012-10-11 Vladimir Makarov
* lra-spills.c (spill_pseudos): Process CALL_IN
Hi Mikael,
>> here is a small patch for a wrong-code regression with polymorphic
>> allocation. The problem is that we falsely detect the allocation
>> variable to be a polymorphic array (although it is a scalar). For
>> further details see the PR, in particular comment 4.
>>
>> Regtested on x86_6
The two current users of this field are DECL_FRIEND_CONTEXT for friend
functions and DECL_THUNKS for virtual member functions; I would like to
use it for another purpose for constructors, so I'm changing DECL_THUNKS
to return null for non-virtual functions.
Tested x86_64-pc-linux-gnu, applying
On Wed, 10 Oct 2012, Richard Sandiford wrote:
> > I've thought of -miterative-mdu or suchlike a knob that would override
> > the default cost of multiplication/division as appropriate (i.e. to 32/64
> > plus any extra operation-specific constant as required), perhaps by
> > forcing the 4Kp ins
OK, I've checked this in:
commit 2425dc8daa33d8b5d15c803c9bf2f51f5e1915a0
Author: Jason Merrill
Date: Tue Oct 9 21:16:52 2012 -0400
* g++.dg/ext/visibility/pragma-override1.C: Fix target markup.
* g++.dg/ext/visibility/pragma-override2.C: Fix target markup.
diff --git a/gcc/testsuit
On 10/09/2012 11:27 AM, Jason Merrill wrote:
FAIL: g++.dg/tls/thread_local7.C scan-assembler-not \\.data
I've changed this test to require native TLS.
FAIL: g++.dg/tls/static-1.C *
And I've fixed the compiler to not mess with thread_local wrappers on
this test, since it uses __thread.
Te
On Thu, Oct 11, 2012 at 6:16 AM, Richard Biener
wrote:
> On Thu, Oct 11, 2012 at 3:52 AM, Easwaran Raman wrote:
>> Hi,
>> In the expression reassociation pass, statements might get moved
>> downwards to ensure that dependences are not violated after
>> reassociation. This can increase the live r
The check for .ref support on AIX used the wrong syntax, leading to
broken output from autoconf.
Tested by configuring a cross-compiler, applying to trunk as obvious.
commit 885339b51d3221bcceabeaf9b2e5f39664ad947d
Author: Jason Merrill
Date: Thu Oct 11 09:01:49 2012 -0400
* conf
This patch set has been committed, thanks to all maintainers who
reviewed the respective parts.
Thanks,
Chung-Lin
Full ChangeLog:
2012-10-11 Chung-Lin Tang
* builtins.c (expand_builtin_thread_pointer): New.
(expand_builtin_set_thread_pointer): New.
(expand_builtin):
Pavel Chupin writes:
> Could you try this fix?
Duplicating this much info from libgcc is certainly the wrong (read:
unmaintainable) approach.
Rainer
--
-
Rainer Orth, Center for Biotechnology, Bielefeld Univer
Could you try this fix?
--
Pavel Chupin
Intel Corporation
Fix-missing-gthr-aix.h.patch
Description: Binary data
AIX bootstrap is broken after this change.
This patch also should not have been approved for GCC 4.7 branch
without wider testing on trunk.
Would you please revert this patch while you figure out how to provide
the previous libstdc++ build behavior?
Thanks, David
On Thu, Oct 11, 2012 at 7:32 AM, Jakub Jelinek wrote:
> Hi!
>
> As the shadow memory offset ored into shifted addresses is clearly
> target specific (1 << 29 on i?86, 1LL << 44 on x86_64, 0 on ARM?,
> no support on other target so far), it should be IMHO a target hook.
>
> Ok for asan?
>
> --- gc
Hi,
testcase added to testsuite, PR closed as fixed.
Thanks,
Paolo.
/
2012-10-11 Paolo Carlini
PR c++/36107
* g++.dg/ext/weak5.C: New.
Index: g++.dg/ext/weak5.C
===
--- g++.dg/ext/weak5.C (r
On Thu, Oct 11, 2012 at 4:33 PM, Jakub Jelinek wrote:
> On Thu, Oct 11, 2012 at 04:30:42PM +0200, Richard Biener wrote:
>> > The comment wants to say that, but doesn't do it correctly:
>> >Operand 0 is a vector; the first element in the vector has the result.
>> >Operand 1 is a vector. */
On Thu, Oct 11, 2012 at 11:25 AM, Marc Glisse wrote:
> Hello,
>
> this documents both my latest C++ FE patch, something I noticed while doing
> the FE work (shifts can mix vectors and scalars) and the result of the
> conversation with Richard B on how the middle-end should model vector truth
> val
On Thu, Oct 11, 2012 at 04:30:42PM +0200, Richard Biener wrote:
> > The comment wants to say that, but doesn't do it correctly:
> >Operand 0 is a vector; the first element in the vector has the result.
> >Operand 1 is a vector. */
> > because obviously it doesn't have two operands, just on
Hi!
As the shadow memory offset ored into shifted addresses is clearly
target specific (1 << 29 on i?86, 1LL << 44 on x86_64, 0 on ARM?,
no support on other target so far), it should be IMHO a target hook.
Ok for asan?
2012-10-11 Jakub Jelinek
* toplev.c (process_options): Warn and t
On Thu, Oct 11, 2012 at 3:47 PM, Jakub Jelinek wrote:
> On Thu, Oct 11, 2012 at 03:42:44PM +0200, Richard Biener wrote:
>> On Wed, Oct 10, 2012 at 7:49 PM, Jakub Jelinek wrote:
>> > This patch folds REDUC_*_EXPR (e.g. on pr54877.c -Ofast -mavx
>> > testcase we end up with unfolded REDUC_PLUS_EXPR
On Thu, 11 Oct 2012, Jan Hubicka wrote:
> Hi,
> this patch address problem I run into with strenghtened cunroll pass. I made
> cunroll to use loop_max_iterations bounds into an account that makes us to
> occasionally produce out of bounds loop accesses in loop like:
>
> for (i=0;i {
>
> if
On 10/10/2012 08:41 PM, Cary Coutant wrote:
I'm working on a follow-up patch to eliminate the duplicate references
to ".LVL" symbols by keeping a direct-lookup table in
dwarf2out_var_location. That will eliminate 310,266 of the 312,061
duplicates (99.4%) without using a hash table. With that patc
Hi,
this patch address problem I run into with strenghtened cunroll pass. I made
cunroll to use loop_max_iterations bounds into an account that makes us to
occasionally produce out of bounds loop accesses in loop like:
for (i=0;i
if (test)
break
a[i]=1;
}
Here the constantly sized array a
On Thu, Oct 11, 2012 at 03:42:44PM +0200, Richard Biener wrote:
> On Wed, Oct 10, 2012 at 7:49 PM, Jakub Jelinek wrote:
> > This patch folds REDUC_*_EXPR (e.g. on pr54877.c -Ofast -mavx
> > testcase we end up with unfolded REDUC_PLUS_EXPR till *.optimized).
> >
> > Bootstrapped/regtested on x86_64
As discussed earlier, this patch renames tree-asan.[ch] to asan.[ch].
Tested on x86_64. Committed to branch.
Diego.
* asan.c: Rename from tree-asan.c.
Update all users.
* asan.h: Rename from tree-asan.h
Update all users.
On Wed, Oct 10, 2012 at 7:49 PM, Jakub Jelinek wrote:
> Hi!
>
> This patch folds REDUC_*_EXPR (e.g. on pr54877.c -Ofast -mavx
> testcase we end up with unfolded REDUC_PLUS_EXPR till *.optimized).
>
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
Eh, I didn't realize the res
On Thu, 11 Oct 2012, Richard Biener wrote:
On Thu, Oct 11, 2012 at 2:19 PM, Marc Glisse wrote:
Hello,
understanding the testsuite is hard, but this patch seems to work. The idea
is that vect_double and others don't tell that vectors are supported with
these options, just that there exist opti
On Thu, Oct 11, 2012 at 2:19 PM, Marc Glisse wrote:
> Hello,
>
> understanding the testsuite is hard, but this patch seems to work. The idea
> is that vect_double and others don't tell that vectors are supported with
> these options, just that there exist options that enable vectors. They are
> th
On Thu, Oct 11, 2012 at 3:52 AM, Easwaran Raman wrote:
> Hi,
> In the expression reassociation pass, statements might get moved
> downwards to ensure that dependences are not violated after
> reassociation. This can increase the live range and, in a tight loop,
> result in spills. This patch sim
Hello,
The change apparently broke targets that have their gthr- header not
in libgcc/ directly, quite a few.
We are seeing failure on ppc-aix for example, which has
./libgcc/config/rs6000/gthr-aix.h
We are also seeing failures while building a win32 canadian
from linux, where we have
./l
On Thu, Oct 11, 2012 at 7:31 AM, Xinliang David Li wrote:
> On Fri, Oct 5, 2012 at 1:49 AM, Richard Guenther
> wrote:
>> On Thu, Oct 4, 2012 at 8:16 PM, Diego Novillo wrote:
>>> On Thu, Oct 4, 2012 at 2:14 PM, Lawrence Crowl wrote:
>>>
So, Jan Hubicka requested and approved the current spe
Hi Richard,
On 21 September 2012 10:49, Richard Earnshaw wrote:
> On 21/09/12 09:47, Matthew Gretton-Dann wrote:
>> On 20 September 2012 23:06, Christophe Lyon
>> wrote:
>>> Hi,
>>>
>>> GCC for ARM does not support compiling in Thumb1 mode and
>>> float-abi=hard. But it does not fail unless
> 2012-10-09 Eric Botcazou
> PR bootstrap/54820
> * Makefile.tpl (STAGE1_FLAGS_TO_PASS): New variable.
> (host_modules): Add STAGE1_FLAGS_TO_PASS to args.
> * Makefile.in: Regenerate.
> * configure.ac (have_static_libs): New variable and associated check.
>
On Thu, Oct 11, 2012 at 1:20 AM, Marc Glisse wrote:
> On Mon, 8 Oct 2012, Richard Guenther wrote:
>
>> On Mon, Oct 8, 2012 at 11:34 AM, Marc Glisse wrote:
>>>
>>> On Mon, 8 Oct 2012, Richard Guenther wrote:
>>>
> VEC_COND_EXPR is more complicated. We could for instance require that
> it
>
Hi
contributed by Daniel. Tested again and committed.
Paolo.
///
2012-10-11 Daniel Krugler
* testsuite/20_util/common_type/requirements/sfinae_friendly_1.cc:
Update / extend.
* testsuite/20_util/common_type/requirements/sfinae_friendly_2.cc:
Hello,
understanding the testsuite is hard, but this patch seems to work. The
idea is that vect_double and others don't tell that vectors are supported
with these options, just that there exist options that enable vectors.
They are thus useless outside of a directory like vect/ that adds those
On Thu, 11 Oct 2012, Rainer Orth wrote:
> Ok, here's the merged patch, tested on i386-pc-solaris2.10. Ok for
> mainline?
Cool, thank you!
Just a small note, in the following
+#ifdef __FreeBSD__
+# define DEFAULT_PROCESS_FILENAME "/proc/curproc/file"
+#elif defined(HAVE_GETEXECNAME)
+#
Hello!
For 32bit targets, there is no guarantee that mxcsr register will be
set up correctly w.r.t. unmasked FP exceptions, so don't use SSE
instructions to generate INVALID and DIVZERO FP exceptions.
2012-10-10 Uros Bizjak
* config/i386/sfp-exceptions.c (__sfp_handle_exceptions): Emi
Ian Lance Taylor writes:
> On Fri, Oct 5, 2012 at 4:21 AM, Rainer Orth
> wrote:
>>
>> I think this should work. The only complication might be that users of
>> DEFAULT_PROCESS_FILENAME need to include on Solaris to get
>> the declaration of getexecname(). Given that this header is already
>>
> From: Hans-Peter Nilsson
> Date: Thu, 11 Oct 2012 02:13:32 +0200
> There's now an "excess error":
>
> x/libstdc++-v3/testsuite/23_containers/bitset/45713.cc:24:55:
> error: size of array 'test' is not an integral constant-expression
> int test[sizeof(std::bitset<0x>) != 1 ? 1 : -1];
>
> 2012-10-09 Eric Botcazou
>
> * c-ada-spec.c (dump_ada_template): Bail out for template declarations
> declaring something coming from another file.
That's a small patch affecting only -fdump-ada-spec so I went ahead and
installed it on the mainline.
--
Eric Botcazou
Hi,
I'm committing the testcase and closing the PR as fixed in mainline.
Thanks,
Paolo.
///
2012-10-11 Paolo Carlini
PR c++/43765
* g++.dg/parse/pr43765.C: New.
Index: g++.dg/parse/pr43765.C
The new scan-rtl-dump-not I recently added needs to be tweaked so as to cater
to x86-64 platforms like Solaris where the prologue isn't elided.
Tested on x86_64-suse-linux, applied on the mainline as obvious.
2012-10-11 Eric Botcazou
PR testsuite/54867
* gcc.dg/pr44194-1.c:
Hello,
this documents both my latest C++ FE patch, something I noticed while
doing the FE work (shifts can mix vectors and scalars) and the result of
the conversation with Richard B on how the middle-end should model vector
truth values. The patch is a little early compared to the code
(VEC_C
1 - 100 of 104 matches
Mail list logo
