On 02/18/13 15:32, Jakub Jelinek wrote:
Hi!
This accesses one past the last initialized argument.
Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for
trunk?
2013-02-18 Jakub Jelinek
PR tree-optimization/56381
* tree-ssa-pre.c (create_expression_by_pie
> > >> AFAICT, for gcc+gas it should already work with binutils that include
> > >> the AdaCore patch for DWARF support. But this has apparently not been
> > >> tested with AIX ld, and there are AdaCore local patches pending.
> > >> http://sourceware.org/ml/binutils/2011-04/msg00250.html
> > >> htt
On Mon, Feb 18, 2013 at 12:50:59PM -0600, Aldy Hernandez wrote:
> OK pending tests?
> PR target/52555
> * genopinit.c (raw_optab_handler): Use this_fn_optabs.
> (swap_optab_enable): Same.
> (init_all_optabs): Use argument instead of global.
> * tree.h (struct tree_opt
On 27 March 2012 21:25, Jonathan Wakely wrote:
>* include/std/functional (mem_fn): Qualify to prevent ADL.
>* testsuite/20_util/function_objects/mem_fn/adl.cc: New.
>
> Tested x86_64-linux, committed to trunk. Not a regression but should
> be safe for all branches.
Attached patch
On 19 November 2012 21:35, Jonathan Wakely wrote:
> * include/bits/hashtable.h: Improve comments.
> * include/bits/hashtable_policy.h: Likewise.
>
> Tested x86_64-linux, committed to trunk.
Attached version committed to the 4.7 branch.
commit 8db4c698a78df8c150ea3353aaed254448620fb
On 16 January 2013 23:57, Jonathan Wakely wrote:
> Here's another attempt to fix this regression, I hope this time it
> doesn't cause more problems than it solves.
>
> Instead of specializing is_copy_constructible for the unordered
> containers this causes their copy constructors to be deleted if t
Hi!
This accesses one past the last initialized argument.
Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for
trunk?
2013-02-18 Jakub Jelinek
PR tree-optimization/56381
* tree-ssa-pre.c (create_expression_by_pieces): Fix up last argument
to fold
On Mon, Feb 18, 2013 at 10:29 PM, Steven Bosscher wrote:
>> /home/stevenb/lra_powerpc64_autotester/lra-branch/libgfortran/generated/pow_i8_i4.c:
>> In function 'pow_i8_i4':
>> /home/stevenb/lra_powerpc64_autotester/lra-branch/libgfortran/generated/pow_i8_i4.c:75:1:
>> internal compiler error: in c
Tilman Sauerbeck [2013-02-18 22:47]:
> [...]
> + "TARGET_32BIT"
> + "@
> + tst%?\\t%0, %1"
> [...]
I managed to post the wrong diff -- line 2 in the citation should be
omitted of course. Sorry.
Regards,
Tilman
--
A: Because it messes up the order in which people normally read text.
Q: Why
Hi,
adding the instruction pattern below fixes my testcase for PR 56110;
however I'm not sure if adding a new pattern is the correct way to go.
I duplicated the andsi3_compare0_scratch pattern, and lifted the
requirement that the 2nd operand be an arm_not_operand. I didn't copy
over the clobber pa
On Mon, Feb 18, 2013 at 9:21 PM, Steven Bosscher wrote:
> On Sat, Feb 16, 2013 at 1:55 AM, Vladimir Makarov wrote:
>> The branch was merged with trunk @ r196091.
>>
>> Successfully bootstrapped on x86-64.
>>
>> Committed as rev. 196099.
>
> Hello Vlad,
>
> It looks like this broke bootstrap on ia64
On Sat, Feb 16, 2013 at 1:55 AM, Vladimir Makarov wrote:
> The branch was merged with trunk @ r196091.
>
> Successfully bootstrapped on x86-64.
>
> Committed as rev. 196099.
Hello Vlad,
It looks like this broke bootstrap on ia64 and
/home/stevenb/lra_powerpc64_autotester/lra-branch/libgfortran/g
Hi,
after much pondering about PR 55334 I came to conclusion that no nice
fix to the regression could be introduced in stage4. So for the sake
of the SPEC 200 benchmark I decided to cripple IPA-CP on restrict
pointers to arrays so that the restrict-ness of the memory references
which need to be v
Hi,
in ipa-cp there is a dumping condition that was intended to trigger
for non-aliases and non-thunks but I have lost the negations somewhere
when introducing the code. This probably shows this particular
dumping is not very crucial but I think it summarizes the
initialization well and decided t
On Mon, Feb 18, 2013 at 7:44 PM, Igor Zamyatin wrote:
> This small change enables FP reassociation for AVX2 processors. This
> gives ~+1.5% in performance geomean for spec2006FP tests.
>
> Bootstrapped/regtested on x86_64-linux, ok for trunk?
>
>
> 2013-02-14 Igor Zamyatin
>
> * config
On Mon, Feb 18, 2013 at 12:23 AM, Matthias Klose wrote:
>
>> As I mentioned on the bug, can you figure out why it fails? The file
>> gcc/testsuite/lib/go.exp does a load_lib of several files from that
>> directory. Why would they all work but loading timeout.exp fails?
>
> but as I see now, it s
Hi!
On the following testcase, a = a / 8; looks like reasonable reduction
statement, but we pattern recognize it as
patt_1 = a < 0 ? 7 : 0;
patt_2 = a + patt_1;
a = patt2 >> 3;
and in the first pattern stmt the reduction operand is in the condition of
COND_EXPR, not operands as vectorizable_reduct
On 02/16/13 05:19, Richard Sandiford wrote:
Looks good to me otherwise, thanks.
Implemented all your suggestions.
Sorry that SWITCHABLE_TARGETS has been so much hassle. TBH, like Jakub
says in the PR, I was hoping things like the optimize attribute could
use the target_globals stuff too.
This patch adds a comment explaining why it is difficult to split Thumb1
patterns.
gcc/
2013-02-12 Greta Yorsh
* config/arm/arm.md: Comment on splitting Thumb1 patterns.diff --git a/gcc/config/arm/arm.md b/gcc/config/arm/arm.md
index 64888f9..ce98013 100644
--- a/gcc/config/arm/arm.md
+++
Convert define_insn into define_insn_and_split for various min and max
patterns that output multiple assembly instructions. Use movsicc to emit
RTL. A separate patch will split movsicc.
gcc/
2013-02-14 Greta Yorsh
* config/arm/arm.md (arm_smax_insn): Convert define_insn into
def
Hi!
This small change enables FP reassociation for AVX2 processors. This
gives ~+1.5% in performance geomean for spec2006FP tests.
Bootstrapped/regtested on x86_64-linux, ok for trunk?
2013-02-14 Igor Zamyatin
* config/i386/i386.c (initial_ix86_tune_features): Turn on fp
rea
Convert define_insn into define_insn_and_split for various DImode shift
operations that output multiple assembly instructions.
This patch also adds a new pattern for RRX using a new UNSPEC. This pattern
matches RTL insns emitted by arm_ashrdi3_1bit and arm_lshrdi3_1bit
splitters. This patch also a
This patch adds patterns to handle negation of an extended 32-bit value more
efficiently.
For example,
(set (reg:DI r0) (neg:DI (sign_extend:DI (reg:SI r0)))
The compiler currently generates
mov r1, r0, asr #31
rsbsr0, r0, #0
rsc r1, r1, #0
and after the patch
Convert define_insn into define_insn_and_split for various patterns that
output multiple assembly instructions.
It appears that preparation statements in define_insn_and_split sometimes
are called with which_alternative set to -1 even after reload. Therefore,
preparation statements use conditions
Convert define_insn into define_insn_and_split for various subdi patterns
that output multiple assembly instructions.
2013-02-14 Greta Yorsh
* config/arm/arm.md (arm_subdi3): Convert define_insn into
define_insn_and_split.
(subdi_di_zesidi,subdi_di_sesidi): Likewi
Add patterns to handle various subtract with carry operations.
These patterns match RTL insns emitted by splitters
for DImode operations such as subdi, negdi, and cmpdi.
gcc/
2013-02-14 Greta Yorsh
* config/arm/arm.md (subsi3_carryin, subsi3_carryin_const): New
patterns.
(su
This sequence of patches aims at cleaning up patterns that output multiple
assembly instructions.
The first few patches handle some of the patterns in arm.md.
[1/n] Add new patterns for subtract with carry.
[2/n] Split subdi patterns.
[3/n] Split patterns andsi_iorsi3_notsi, abs, cmpdi, and negd
On 02/15/13 08:08, Jakub Jelinek wrote:
On Thu, Feb 14, 2013 at 05:48:57AM -0700, Jeff Law wrote:
I think two tests should be sufficient. First, compile a simple
program with -g and verify it generates dwarf2 debug records.
Second verify that there aren't any -g options, unless is
dwarf2.
I'm
This patch removes the broken powf hack. This problem is now fixed in
the PA backend.
Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11.
OK for trunk?
Dave
--
J. David Anglin dave.ang...@nrc-cnrc.gc.ca
National Research Council of Canada (613
Am 12.02.2013 13:45, schrieb Richard Biener:
> On Tue, Feb 12, 2013 at 1:44 PM, Richard Biener
> wrote:
>> On Tue, Feb 12, 2013 at 1:30 PM, Matthias Klose wrote:
>>> The libffi library, headers and documentation are still installed, although
>>> libffi provides separate releases for a long time.
The powf function on HP-UX incorrectly clobbers floating point register
%fr12. It is supposed to be saved and restored by callee. The patch
marks the register(s) as call used on HP-UX. We still save and restore
the register is the prologue/epilogue, so the runtime isn't changed.
Tested on hppa2
Hi,
I think submitter is right that with -Wzero-as-null-pointer-constant we
want to warn also for zero converted to nullptr_t, not just pointer
types. In that case the below is the simplest fix I could figure out,
passes testing on x86_64-linux. Otherwise I guess we should simply close
the PR
The attached change fixes PR target/56214. We need to reject everything
but QImode and HImode when reload is in progress and strict is specified.
Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11. Committed to
trunk.
Dave
--
J. David Anglin dave.ang...@nr
The patch successfully disables stabs+PCH on hppa2.0w-hp-hpux11.11 and
fixes the
testsuite regressions. Also tested on hppa64-hp-hpux11.11 with no
regressions.
On 2013-02-15 9:08 AM, Jakub Jelinek wrote:
On Thu, Feb 14, 2013 at 05:48:57AM -0700, Jeff Law wrote:
I think two tests should be su
On Mon, 2013-02-18 at 13:05 +0100, Richard Biener wrote:
> On Fri, Feb 15, 2013 at 6:53 PM, Bill Schmidt
> wrote:
> > When we remove __builtin_pow statements as part of reassociation, we
> > have to unlink the associated VDEF. We've always done this when we
> > directly remove the statement. How
This fixes PR56321 - the unlink_stmt_vdef was a no-op as it was
called after release_defs. Oops.
Committed as obvoious.
Richard.
2013-02-18 Richard Biener
PR tree-optimization/56321
* tree-ssa-reassoc.c (propagate_op_to_single_use): Properly
order SSA name release a
On Fri, Feb 15, 2013 at 6:53 PM, Bill Schmidt
wrote:
> When we remove __builtin_pow statements as part of reassociation, we
> have to unlink the associated VDEF. We've always done this when we
> directly remove the statement. However, in reassociation the statements
> are sometimes modified in p
This fixes another case where missed sign-conversions during vectorization
need to be handled (outer-loop vectorization induction).
Bootstrap and regtest pending on x86_64-unknown-linux-gnu.
Richard.
2013-02-18 Richard Biener
PR tree-optimization/56366
* tree-vect-loop.c (ge
On 01/30/13 09:24, Kyrylo Tkachov wrote:
Hi all,
This patch uses the new ARMv8 AArch32 vsel instruction to implement
conditional moves of floating point numbers.
For example, an instruction of the form:
vsel.f32 s0, s1, s2
means
s0 := cond ? s1 : s2
This can be useful, among oth
The issue in PR56350 is two-fold. First, fix_loop_structure mishandles
latch edge computation (ugh). Second, RTL if-conversion, when merging
a latch block into sth else ends up generating stale latch block
references.
The following two patches cure that, adding some basic latch block
checking.
Committed as r196116.
On Sun, 17 Feb 2013, Gerald Pfeifer wrote:
> On Thu, 31 Jan 2013, Richard Biener wrote:
> > +GCC now uses more a aggressive analysis to derive an upper bound for
>
> I think "a " should be omitted here.
Yes, that was fixed before commit.
> > +the number of iterations of loops using constraints
On Sun, 17 Feb 2013, Gerald Pfeifer wrote:
> On Thu, 7 Feb 2013, Richard Biener wrote:
> > As promised, a bugs.html entry.
>
> Looks good to me.
>
> Now, at the top of the page we have
>
> Before reporting that GCC compiles your code incorrectly, compile it
> with gcc -Wall -Wextra and see
On Sat, 16 Feb 2013, Dodji Seketeli wrote:
> Jakub Jelinek writes:
>
> > On Sat, Feb 16, 2013 at 10:40:43AM +0100, Dodji Seketeli wrote:
> >> --- a/gcc/cfg.c
> >> +++ b/gcc/cfg.c
> >> @@ -663,6 +663,21 @@ debug_bb_n (int n)
> >>return bb;
> >> }
> >>
> >> +/* Dumps cfg related information
On 02/12/13 19:47, Lawrence Crowl wrote:
Add class record_builder to ease construction of records and unions. Use it
in some appropriate places.
Nathan please review the vxworks changes.
config/vxworks.c
Replace vxworks_emutls_var_fields() with vxworks_emutls_object_type().
U
On Mon, Feb 18, 2013 at 12:20 PM, Jakub Jelinek wrote:
> On Fri, Feb 15, 2013 at 07:39:28AM -0800, Ian Lance Taylor wrote:
>> On Thu, Feb 14, 2013 at 11:45 PM, Konstantin Serebryany
>> wrote:
>> >
>> > Unfortunately, the test does not work if gold is the system linker.
>> > Any suggestion on how
Am 09.02.2013 19:24, schrieb Ian Lance Taylor:
> On Sat, Feb 9, 2013 at 3:18 AM, Andreas Schwab wrote:
>> Ian Lance Taylor writes:
>>
>>> 2013-02-08 Ian Lance Taylor
>>>
>>> * lib/go.exp: Load timeout.exp.
>>
>> ERROR: Couldn't find library file timeout.exp.
>
> Sorry, I was told on IRC
On Fri, Feb 15, 2013 at 07:39:28AM -0800, Ian Lance Taylor wrote:
> On Thu, Feb 14, 2013 at 11:45 PM, Konstantin Serebryany
> wrote:
> >
> > Unfortunately, the test does not work if gold is the system linker.
> > Any suggestion on how to make the test work with either linker?
>
> I don't know of
48 matches
Mail list logo
