Re: [PATCH, i386]: Macroize min/max patterns some more

2011-09-21 Thread Jakub Jelinek
On Mon, Sep 19, 2011 at 07:10:11PM +0200, Uros Bizjak wrote: > 2011-09-19 Uros Bizjak > > * config/i386/i386.md (maxmin): New code iterator. > * config/i386/sse.md (3): Macroize expander > from 3 and 3 using maxmin > code iterator. > (*avx2_3): Macroize isn from >

Re: [PATCH, i386]: Macroize min/max patterns some more

2011-09-21 Thread Uros Bizjak
On Wed, Sep 21, 2011 at 9:45 AM, Jakub Jelinek wrote: >>       * config/i386/i386.md (maxmin): New code iterator. >>       * config/i386/sse.md (3): Macroize expander >>       from 3 and 3 using maxmin >>       code iterator. >>       (*avx2_3): Macroize isn from >>       *avx2_3 and *avx2_3 usin

[arm-embedded] Tune branch cost for armv7-m

2011-09-21 Thread Joey Ye
Committed in ARM/embedded-4_6-branch. 2011-09-20 Jiangning Liu Tune branch cost for armv7-m * config/arm/arm-cores.def (cortex-m3): Use cortex-v7m. (cortex-m4) : Likewise. * config/arm/arm.c (arm_cortex_v7m_branch_cost): New. (arm_cortex_v7m_tune): New.

[PATCH, testsuite] Add loop unrolling command line options for some test cases

2011-09-21 Thread Jiangning Liu
Hi, The following test cases are to check predictive commoning optimization by detecting loop unrolling times. Originally by default the max-unroll-times is 8. If max-unroll-times is specified to be less than the expected unrolling times, those cases would fail. The fix is to explicitly turn on

[PATCH] Use TREE_READONLY on some sparc VIS builtins

2011-09-21 Thread David Miller
While fiddling around with the VIS intrinsic builtins I noticed that none of them have TREE_READONLY set, so the resulting code can be terrible. We can't currently do this for alignaddr and faligndata because we don't model the way those instructions use the %gsr register. Committed to trunk. g

[arm-embedded] Tune loop unrolling for cortex-m

2011-09-21 Thread Joey Ye
Committed in ARM/embedded-4_6-branch. 2011-09-21 Jiangning Liu Tune loop unrolling for cortex-m * config/arm/arm-cores.def (cortex-m0): Change to new tune cortex_v6m. (cortex-m1): Likewise. * config/arm/arm-protos.h (max_unroll_times): New. * con

Re: [Mingw-users] [patch] --enable-dynamic-string default for mingw-w64

2011-09-21 Thread xunxun
于 2011/9/21 10:56, Cesar Strauss 写道: Please let me present an opposing view. On the one hand, according to comment 4 of [1], by using --enable-fully-dynamic-string, all other users will miss a very good optimization. On the other hand, these users of -static-libstdc++ are mixing shared and st

[PATCH, testsuite, i386] BMI2's intrinsic for MULX instruction

2011-09-21 Thread Kirill Yukhin
Hi there, I've prepared a patch which implements couple of intrinsics for generation of MULX 32 and 64 bit wide (part of BMI2 extensions). Tests are added as well ChangeLog entry: 2011-09-21 Kirill Yukhin * config/i386/bmi2intrin.h (_mulx_u64): New. (_mulx_u32): Ditto. testsui

[wwwdocs] Add info about IPA optimization and LTO improvments

2011-09-21 Thread Jan Hubicka
Hi, this patch adds some info on changes in IPA and LTO. Not sure if I missed something important. Martin, perhaps you could add some example on new devirutalization stuff? Suggestions for less lame explanations are welcome :) Honza Index: changes.html ===

[AVR,committed]: ad PR45099: change error to warning

2011-09-21 Thread Georg-Johann Lay
As proposed in PR45099, avr-gcc will now just print a warning instead of an error when a fixed register is needed to pass a parameter to a function. That way the user can inspect the source, and there are situation imaginable where clobbering a fixed register is not a problem. Committed as trunk

[arm-embedded] Backport mainline r178975

2011-09-21 Thread Terry Guo
Backport mainline r178975 to ARM/embedded-4_6-branch. Committed. 2011-09-21 Terry Guo Backport r178975 from mainline 2011-09-19 Ian Bolton * gcc.dg/20100906-1.c: Use -fno-short-enums option for "target arm_eabi".

Re: [PATCH, testsuite, i386] BMI2's intrinsic for MULX instruction

2011-09-21 Thread Uros Bizjak
On Wed, Sep 21, 2011 at 11:17 AM, Kirill Yukhin wrote: > Hi there, > I've prepared a patch which implements couple of intrinsics for > generation of MULX 32 and 64 bit wide (part of BMI2 extensions). > Tests are added as well > > ChangeLog entry: > 2011-09-21  Kirill Yukhin   > >        * config/i

[v3] Replace everywhere __int128_t -> __int128, and __uint128_t -> unsigned __int128

2011-09-21 Thread Paolo Carlini
Hi, now that c++/50454 is fixed we can do this, replace the legacy typedefs. Tested x86_64-linux multilib, committed. Thanks, Paolo. /// 2011-09-21 Paolo Carlini * src/limits.cc: Replace everywhere __int128_t -> __int128, and __uint128_t -> unsigned __in

[PATCH] Use *blendv* for sse_movcc/vcond/vcondu if possible

2011-09-21 Thread Jakub Jelinek
Hi! For vcond{,u} etc. we currently generate vpandn+vpand+vpor sequence but SSE4.1+ has instructions for at least some modes to handle those 3 in one instruction (haven't benchmarked how much faster/slower it is though). Bootstrapped/regtested on x86_64-linux and i686-linux, tested on SandyBridge

Re: [PATCH] Use *blendv* for sse_movcc/vcond/vcondu if possible

2011-09-21 Thread Uros Bizjak
On Wed, Sep 21, 2011 at 1:37 PM, Jakub Jelinek wrote: > For vcond{,u} etc. we currently generate vpandn+vpand+vpor > sequence but SSE4.1+ has instructions for at least some modes > to handle those 3 in one instruction (haven't benchmarked how much > faster/slower it is though). > > Bootstrapped/r

Re: [PATCH, testsuite, i386] BMI2's intrinsic for MULX instruction

2011-09-21 Thread Kirill Yukhin
> > Comments at #else and #endif are now wrong. Also, please avoid -dp in the > tests. Thanks for inputs. Updated patch is attached (updated comment + updated tests without -dp). New tests still passing Is it OK? Thanks, K bmi2.mulx-intrin-2.gcc.patch Description: Binary data

Re: [PATCH, testsuite, i386] BMI2's intrinsic for MULX instruction

2011-09-21 Thread Uros Bizjak
On Wed, Sep 21, 2011 at 2:07 PM, Kirill Yukhin wrote: >> Comments at #else and #endif are now wrong. Also, please avoid -dp in the >> tests. > > Thanks for inputs. Updated patch is attached (updated comment + > updated tests without -dp). > New tests still passing OK for mainline SVN. Thanks,

Re: MIPS Fix PR18141

2011-09-21 Thread Richard Sandiford
Dinar Temirbulatov writes: > @@ -14696,7 +14696,11 @@ mips_avoid_hazard (rtx after, rtx insn, > clobber hi and lo. */ >if (*hilo_delay < 2 && reg_set_p (lo_reg, pattern)) > nops = 2 - *hilo_delay; > - else if (*delayed_reg != 0 && reg_referenced_p (*delayed_reg, pattern)) > + els

Re: [PATCH] Use *blendv* for sse_movcc/vcond/vcondu if possible

2011-09-21 Thread Jakub Jelinek
On Wed, Sep 21, 2011 at 02:02:44PM +0200, Uros Bizjak wrote: > > +      rtx (*gen) (rtx, rtx, rtx, rtx) = NULL; > > + > >       op_false = force_reg (mode, op_false); > > +      switch (mode) > > +       { > > +       case V4SFmode: if (TARGET_SSE4_1) gen = gen_sse4_1_blendvps; break; > > +       c

Re: [AVR,committed]: ad PR45099: change error to warning

2011-09-21 Thread Nathan Froyd
On 9/21/2011 5:49 AM, Georg-Johann Lay wrote: As proposed in PR45099, avr-gcc will now just print a warning instead of an error when a fixed register is needed to pass a parameter to a function. Where's the proposal in the PR? I see a problem report that was addressed four months ago by addin

Re: inline-analysis improvement

2011-09-21 Thread Jan Hubicka
> > the problem here is that we consider ¶m.foo to be a memory reference, > > while it is not. The following patch should fix it, but because it changes > > the behaviour of inline heuristics, I will run bechmarks tonight before > > committing it. > > Thanks in advance. Can you add PR tree-optimi

Re: [patch tree-optimization]: Improve reassociation pass for bitwise-operations

2011-09-21 Thread Kai Tietz
Hello, This revised patch adds support to tree-ssa-reassoc for intial normalizing of bitwise operation. So the reassociation pass is able to have better folding result. Also it has a final denormalzing of bitwise-operations after reassociation pass is completed to restore optimized writing. This

Re: inline-analysis improvement

2011-09-21 Thread Eric Botcazou
> The tests tonight shows quite good results - there are quite consistent > code size reductions on C++ code w/o slowdowns so I've comitted the patch. > Sadly tramp3d does not build and the heuristics was implemented with > tramp3d in mind. Important behaviour change is that the heuristic now > mat

Re: [PATCH] Use *blendv* for sse_movcc/vcond/vcondu if possible

2011-09-21 Thread Richard Henderson
On 09/21/2011 04:37 AM, Jakub Jelinek wrote: >op_false = force_reg (mode, op_false); > + switch (mode) > + { > + case V4SFmode: if (TARGET_SSE4_1) gen = gen_sse4_1_blendvps; break; > + case V2DFmode: if (TARGET_SSE4_1) gen = gen_sse4_1_blendvpd; break; > + case V16QImod

Re: [PATCH] Use *blendv* for sse_movcc/vcond/vcondu if possible

2011-09-21 Thread Jakub Jelinek
On Wed, Sep 21, 2011 at 07:25:43AM -0700, Richard Henderson wrote: > On 09/21/2011 04:37 AM, Jakub Jelinek wrote: > >op_false = force_reg (mode, op_false); > > + switch (mode) > > + { > > + case V4SFmode: if (TARGET_SSE4_1) gen = gen_sse4_1_blendvps; break; > > + case V2DFmode: i

Re: Handle multi-word regsiters in REG_CFA_RESTORE notes

2011-09-21 Thread Bernd Schmidt
On 09/15/11 21:42, Richard Henderson wrote: > On 09/14/2011 06:12 PM, Bernd Schmidt wrote: >> + unsigned int orig_regno = REGNO (reg); >> + int nregs = hard_regno_nregs[orig_regno][GET_MODE (reg)]; >> + while (nregs-- > 0) > > The rest of the file seems to use targetm.dwarf_register_span. > Thi

Re: inline-analysis improvement

2011-09-21 Thread Dominique Dhumieres
Eric, I am not sure to understand the sentence: > There is apparently an ACATS failure on x86-64/Darwin, but I've installed the > testcase as gnat.dg/opt19.adb in the tree. The only failure I had was pr50433, now fixed by revision 179046. With it, all the ADA tests pass (including gnat.dg/opt1

Re: inline-analysis improvement

2011-09-21 Thread Iain Sandoe
On 21 Sep 2011, at 15:58, Dominique Dhumieres wrote: I am not sure to understand the sentence: There is apparently an ACATS failure on x86-64/Darwin, but I've installed the testcase as gnat.dg/opt19.adb in the tree. The only failure I had was pr50433, now fixed by revision 179046. With

Re: [ARM] Fix a performance regression from the fix for PR49030

2011-09-21 Thread Ramana Radhakrishnan
On 20 September 2011 09:38, Richard Sandiford wrote: > My fix for PR49030 had the unintended side-effect of forcing libcalls > to be used for some DImode comparisons.  The problem (which I should > have noticed at the time, sorry) is that arm_comparison_operator is > used for both expanders and in

[C++ Patch] PR 50344

2011-09-21 Thread Paolo Carlini
Hi, basing on [class.friend] (11.3/3) in C++11 I think Jon is right that cv-qualification should be ok in a friend declaration. I'm not sure, however, if we want to somehow allow that also in c++98 mode... Tested x86_64-linux. Ok? Thanks, Paolo. /// /cp 2011-09-21 Paolo Ca

Re: [PATCH] Use *blendv* for sse_movcc/vcond/vcondu if possible

2011-09-21 Thread Richard Henderson
On 09/21/2011 07:40 AM, Jakub Jelinek wrote: > * config/i386/i386.c (ix86_expand_sse_movcc): Use > blendvps, blendvpd and pblendvb if possible. > > * gcc.dg/vect/vect-cond-7.c: New test. > * gcc.target/i386/sse4_1-cond-1.c: New test. > * gcc.target/i386/avx-cond-1.c:

[PATCH] reload: Fix problem with CONST_INT only addresses

2011-09-21 Thread Andreas Krebbel
Hi, with the attached testcase I run into: if (this_insn_is_asm) { enum machine_

Re: [PATCH, testsuite, i386] BMI2's intrinsic for MULX instruction

2011-09-21 Thread H.J. Lu
On Wed, Sep 21, 2011 at 5:16 AM, Uros Bizjak wrote: > On Wed, Sep 21, 2011 at 2:07 PM, Kirill Yukhin > wrote: > >>> Comments at #else and #endif are now wrong. Also, please avoid -dp in the >>> tests. >> >> Thanks for inputs. Updated patch is attached (updated comment + >> updated tests without

Re: [Patch][Cilkplus] Fix inling in function calls that spawn

2011-09-21 Thread H.J. Lu
On Tue, Sep 20, 2011 at 7:51 PM, Iyer, Balaji V wrote: > Hello Everyone, >    This patch is for the Cilkplus branch. In C and C++ compilers, > at optimization -O and -O2, when there are inlined functions inside a > function that calls a spawn it was giving an error. This patch should

[Patch,AVR]: Fix PR50447 (1/n)

2011-09-21 Thread Georg-Johann Lay
This patch improves IOR, XOR, AND for HI and SI. There is room for improvement in the current implementation: - better analysis if a scratch is actually needed - add clobber operands to some insns - use 8-bit scratch instead of reload the constant - reuse value in scratch register if known - remo

Re: [PATCH, testsuite, i386] BMI2's intrinsic for MULX instruction

2011-09-21 Thread Kirill Yukhin
Thank you! K On Wed, Sep 21, 2011 at 7:57 PM, H.J. Lu wrote: > On Wed, Sep 21, 2011 at 5:16 AM, Uros Bizjak wrote: >> On Wed, Sep 21, 2011 at 2:07 PM, Kirill Yukhin >> wrote: >> Comments at #else and #endif are now wrong. Also, please avoid -dp in the tests. >>> >>> Thanks for inpu

Re: [AVR,committed]: ad PR45099: change error to warning

2011-09-21 Thread Georg-Johann Lay
Nathan Froyd wrote: > On 9/21/2011 5:49 AM, Georg-Johann Lay wrote: >> As proposed in PR45099, avr-gcc will now just print a warning instead >> of an >> error when a fixed register is needed to pass a parameter to a function. > > Where's the proposal in the PR? I see a problem report that was > a

Re: [C++ Patch] PR 50344

2011-09-21 Thread Jason Merrill
On 09/21/2011 11:12 AM, Paolo Carlini wrote: basing on [class.friend] (11.3/3) in C++11 I think Jon is right that cv-qualification should be ok in a friend declaration. I'm not sure, however, if we want to somehow allow that also in c++98 mode... I think we do. Jason

Re: Handle multi-word regsiters in REG_CFA_RESTORE notes

2011-09-21 Thread Richard Henderson
On 09/21/2011 07:55 AM, Bernd Schmidt wrote: > On 09/15/11 21:42, Richard Henderson wrote: >> On 09/14/2011 06:12 PM, Bernd Schmidt wrote: >>> + unsigned int orig_regno = REGNO (reg); >>> + int nregs = hard_regno_nregs[orig_regno][GET_MODE (reg)]; >>> + while (nregs-- > 0) >> >> The rest of the

Ping [ARM back-end and middle-end patch] stack check for threads

2011-09-21 Thread Thomas Klein
ping rename subject line due to http://gcc.gnu.org/ml/gcc-patches/2011-09/msg01189.html references http://gcc.gnu.org/ml/gcc-patches/2011-09/msg00310.html http://gcc.gnu.org/ml/gcc-patches/2011-08/msg00216.html http://gcc.gnu.org/ml/gcc-patches/2011-07/msg00281.html http://gcc.gnu.org/ml/gcc-pat

[PATCH, i386]: Fix PR50464, ICE with XOP pcmov insn

2011-09-21 Thread Uros Bizjak
Hello! Predicates for xop_pcmov were wrong. Also, immediate operand can leak in through vcond patterns, so a fixup is needed in ix86_expand_sse_movcc for TARGET_XOP. 2011-09-21 Uros Bizjak PR target/50464 * config/i386/sse.md (xop_pcmov_): Change operand 1 predicate to

Re: [google] Backport r178995 from trunk to google/gcc-4_6 (issue5075046)

2011-09-21 Thread Diego Novillo
On Wed, Sep 21, 2011 at 02:24, Guozhi Wei wrote: > 2011-09-21  Guozhi Wei   > >        Backport r178995 from trunk. > >        2011-09-20  Wei Guozhi   > >                PR rtl-optimization/49452 >                * postreload.c (reload_combine): Invalidate use information > when >              

Go patch committed: Support nil maps

2011-09-21 Thread Ian Lance Taylor
The Go language spec clarified that a lookup on nil map should work and return a zero value. This patch to the Go frontend and the Go library implements that. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. (This patch requires rebuilding the entire Go libr

Re: [Patch] PR c++/26256

2011-09-21 Thread Fabien Chêne
Hi, 2011/9/20 Jason Merrill : > On 09/17/2011 09:44 AM, Fabien Chêne wrote: >> >> I tried various things without success, and I ended up hacking >> supplement_binding_1 to handle those ENUMERAL_TYPEs. >> I am all ear for another solution ... > > Your solution seems reasonable to me, but it needs a

Re: [Patch] PR c++/26256

2011-09-21 Thread Fabien Chêne
... with the ChangeLog gcc/ChangeLog 2011-09-21 Fabien Chêne PR c++/26256 * dbxout.c (dbxout_type_fields): Ignore using declarations. gcc/testsuite/ChangeLog 2011-09-21 Fabien Chêne PR c++/26256 * g++.dg/lookup/using23.C: New. * g++.dg/lookup/usi

Re: [Patch] PR c++/26256

2011-09-21 Thread Jason Merrill
On 09/21/2011 01:59 PM, Fabien Chêne wrote: if (!DECL_DEPENDENT_P (field)) - continue; + { + tree using_decl = USING_DECL_DECLS (field); + if ((TREE_CODE (using_decl) == FIELD_DECL + || TREE_CODE (using_decl) == TYPE_DECL) +&&

Re: [C++ Patch] PR 50344

2011-09-21 Thread Paolo Carlini
On 09/21/2011 06:55 PM, Jason Merrill wrote: I think we do. Agreed. I'm finishing testing the below, Ok if it passes? Thanks, Paolo. // /cp 2011-09-21 Jonathan Wakely Paolo Carlini PR c++/50344 * friend.c (make_friend_class): cv-qualifi

Re: [PATCH 3/7] Emit macro expansion related diagnostics

2011-09-21 Thread Dodji Seketeli
Jason Merrill writes: > My point was more that the name of the function is confusing; Sorry about that. > if what you get back is another virtual location, that's not what I > would consider a "def point". For tokens that are not function-like macro arguments, I think the linemap_macro_loc_to_

Re: [pph] Fix pph_read_tree_header. (issue 5050045)

2011-09-21 Thread dnovillo
http://codereview.appspot.com/5050045/diff/1/gcc/cp/pph-streamer-in.c File gcc/cp/pph-streamer-in.c (right): http://codereview.appspot.com/5050045/diff/1/gcc/cp/pph-streamer-in.c#newcode2024 gcc/cp/pph-streamer-in.c:2024: /* Read the language-independent bitfields for expr. */ s/expr/EXPR/ htt

[trunk] contribute GDB/thread testing infrastructure from cxx-mem-model branch

2011-09-21 Thread Aldy Hernandez
I've gotten tired of the false positives in the current testsuite for the cxxbitfield* tests. The tests are highly dependent on assembly output, and currently they only work on x86-64/x86-32 (mostly). I have rewritten the tests to be platform agnostic, by using the GDB/thread testing infrastr

Re: [PATCH, testsuite] Add loop unrolling command line options for some test cases

2011-09-21 Thread Mike Stump
On Sep 21, 2011, at 1:22 AM, Jiangning Liu wrote: > The fix is to explicitly turn on loop unroll and set max-unroll-times to 8, > which is larger than the unrolling times being detected in the cases. Sounds reasonable to me. Ok, though, do watch for any comments by people that know more than I.

Re: [pph] Stream merging information (issue 5090041)

2011-09-21 Thread dnovillo
http://codereview.appspot.com/5090041/diff/1/gcc/cp/pph-streamer-in.c File gcc/cp/pph-streamer-in.c (right): http://codereview.appspot.com/5090041/diff/1/gcc/cp/pph-streamer-in.c#newcode2146 gcc/cp/pph-streamer-in.c:2146: pph_read_namespace_chain (pph_stream *stream, tree enclosing_namespace) 2

Re: [google] Add new warning -Wreal-conversion (issue4436068)

2011-09-21 Thread Diego Novillo
On Tue, May 3, 2011 at 16:20, Jason Merrill wrote: > On 04/29/2011 10:59 AM, Diego Novillo wrote: >> >> This patch from Le-Chun Wu adds a new warning flag "-Wreal-conversion" >> that warns about implicit type conversions from real (double or float) >> values to integral values. > > Why do you want

Go patch committed: Fix shift expressions

2011-09-21 Thread Ian Lance Taylor
This patch to the Go frontend fixes a couple of cases with shift expressions. A shift of a floating point constant (which would have to have integral value) is still a constant, and as such must be lowered to a constant value. This patch fixes the general case of shifting a non-numeric value, whe

[Patch, Fortran] PR 41733: Proc-pointer conformance checks: Elemental-proc-ptr => non-elemental-proc

2011-09-21 Thread Janus Weil
Hi all, here is a patch which adds checks for the ELEMENTAL attribute in procedure pointer assignments and dummy procedures. For details see the PR and the c.l.f. thread mentioned therein. For the PURE attribute, we already had a check, which I moved and reformulated (so that it is applicable also

Re: [Patch, Fortran] PR 41733: Proc-pointer conformance checks: Elemental-proc-ptr => non-elemental-proc

2011-09-21 Thread Steve Kargl
On Wed, Sep 21, 2011 at 10:27:34PM +0200, Janus Weil wrote: > Hi all, > > here is a patch which adds checks for the ELEMENTAL attribute in > procedure pointer assignments and dummy procedures. For details see > the PR and the c.l.f. thread mentioned therein. For the PURE > attribute, we already ha

Re: [RFC] Add FMA support to sparc backend

2011-09-21 Thread David Miller
From: Michael Meissner Date: Thu, 15 Sep 2011 14:16:45 -0400 > On the powerpc, we have an issue with Spec 2006 and calculix when FMAs are > generated and -ffast-math is used, where line 307 of rubber.f is: > > tt=datan2(dsqrt(1.d0-cn*cn),cn)/3.d0 > > The FNMSUB instruction generates a -0.

[PATCH, testsuite]: Remove unneeded dg-require-effective-target sse directives

2011-09-21 Thread Uros Bizjak
Hello! Compile tests do not need to check assembler for sse support. 2011-09-21 Uros Bizjak * g++.dg/other/i386-3.C: Remove duplicated dg-options. * g++.dg/other/i386-4.C: Remove dg-requre-effective-target directive. * g++.dg/other/i386-8.C: Ditto. * g++.dg/oth

Go patch committed: Better handling of EOF

2011-09-21 Thread Ian Lance Taylor
This small patch improves the handling of one case of unexpected EOF in the Go frontend parser. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 5a1ab3177e67 go/parse.cc --- a/go/parse.cc Wed Sep 21 13:20:32 2011 -0700 +++ b/go/parse.cc Wed Sep 2

Skip some x86 tests for conflicting -march= options

2011-09-21 Thread Joseph S. Myers
Similar to various previous such patches, this patch adds dg-skip-if markers to various gcc.target/i386 tests to skip them if the multilib options with which the testsuite is run contain -march= options conflicting with those in the testcases (recall that multilib options go *after* those from dg-o

Fix typos in inliner benefit analysis

2011-09-21 Thread Jan Hubicka
Hi, this patch fixes two rather embarassing typos that together caused that benefits are pretty much ignored by the inliner heuristics. Also to make benefits go right for large function I remived caping logic in do_estimate_edge_time. It is no longer needed since we no longe propagate the values.

Handle non-SSA arguments in ipa-inline-analysis better

2011-09-21 Thread Jan Hubicka
Hi, this patch extends handling of non-SSA arguments to bultin_constant_p and execution predicates. Bootstrapped/regtested x86_64-linux, will commit it shortly. Honza * ipa-inline-analysis.c (set_cond_stmt_execution_predicate): Allow handled components in parameter of builtin_con

Switch function context when doing inline analysis

2011-09-21 Thread Jan Hubicka
Hi, ipa-inline-analysis use is_gimple_min_invariant that in turn require current_function_decl to be set to the corresponding function or all addresses of automatic vars are considered non-invariant. Bootstrapped/regtested x86_64-linux, will commit it shortly. * ipa-inline-analsis.c (c

Go patch committed: Increase default libgo test timeout

2011-09-21 Thread Ian Lance Taylor
I increased the default timeout for the libgo testsuite, simply because the tests were occasionally failing on my old laptop. The tests seem reasonably reliable these days, so the timeout is just a fallback. Most of the tests finish much more quickly. Bootstrapped and ran libgo testsuite on x86_

Re: [3/4] SMS: Record moves in the partial schedule

2011-09-21 Thread Ayal Zaks
Richard Sandiford wrote on 30/08/2011 03:10:50 PM: > From: Richard Sandiford > To: gcc-patches@gcc.gnu.org > Cc: Ayal Zaks/Haifa/IBM@IBMIL > Date: 30/08/2011 03:10 PM > Subject: [3/4] SMS: Record moves in the partial schedule > > This patch adds infrastructure that will be used by the final pat

Re: [SH] AND/OR/XOR costs calculation

2011-09-21 Thread Oleg Endo
> A few minor nits: Thanks for the hints. The attached patch and changelog below should do now. > BTW, have you done your paper work with FSF? > Yep. Took a while but it's done now. Cheers, Oleg 2011-09-21 Oleg Endo * config/sh/sh.c (andcosts): Renamed to and_xor_ior_c

[v3] doc-epub

2011-09-21 Thread Benjamin Kosnik
Fix for image directory location. The generated epub is ok, but missing images. There's another tool that may be used to generate epubs, called db2epub.py, but it isn't cooperating. So we will stick with the status quo for now. -benjamin2011-09-21 Benjamin Kosnik ${manual_xml}; $(XMLLINT) $(

[PATCH] Add pixel compare VIS intrinsics.

2011-09-21 Thread David Miller
Unfortunately, we can't use these for comparisons on vectors as the compiler wants them. Instead of producing a vector of comparison results, these instructions produce a small bitmask of those results in a destination integer register. Therefore we have to provide them using unspecs and builtin

Re: [SH] AND/OR/XOR costs calculation

2011-09-21 Thread Kaz Kojima
Oleg Endo wrote: > * config/sh/sh.c (andcosts): Renamed to and_xor_ior_costs. > Added AND special case. Adapted comments. > (sh_rtx_costs): Added XOR and IOR case. Thanks! I've just applied your patch as revision 179073. Regards, kaz

[PATCH]: Dump the "degree of overlap" to compare static profile with instrumentation profile

2011-09-21 Thread Dehao Chen
Hello! During performance tuning, it's useful to know if the static profile is accurate. This patch compares static profiles to instrumentation profiles, and uses "degree of overlap" to measure its accuracy. Degree of overlap is a number between 0 and 1, representing the correlation between two pr

Re: [PATCH] Add VIS intrinsics header for sparc.

2011-09-21 Thread Hans-Peter Nilsson
On Fri, 16 Sep 2011, David Miller wrote: > > I've been meaning to toss something like this together for a while. > > If we were going to do this, I wanted to get it out of the way before > adding VIS2 and VIS3 support. While revisiting VIS, *please* consider fixing a big usability problem: the pac

Re: [PATCH] Add VIS intrinsics header for sparc.

2011-09-21 Thread David Miller
From: Hans-Peter Nilsson Date: Wed, 21 Sep 2011 21:27:08 -0400 (EDT) > While revisiting VIS, *please* consider fixing a big usability > problem: the pack and aligndata builtins don't take GSR in > account; it has unknown state and might be changed as a > side-effect of a previous VIS insn (well,

Re: [PATCH] Add VIS intrinsics header for sparc.

2011-09-21 Thread Hans-Peter Nilsson
On Wed, 21 Sep 2011, David Miller wrote: > From: Hans-Peter Nilsson > Date: Wed, 21 Sep 2011 21:27:08 -0400 (EDT) > > > While revisiting VIS, *please* consider fixing a big usability > > problem: the pack and aligndata builtins don't take GSR in > > account; it has unknown state and might be chang

RE: [PATCH, testsuite] Add loop unrolling command line options for some test cases

2011-09-21 Thread Jiangning Liu
Hi Mike, OK. I will wait 24 more hours. If no objections by then, I will get it checked into trunk. Thanks, -Jiangning > -Original Message- > From: Mike Stump [mailto:mikest...@comcast.net] > Sent: Thursday, September 22, 2011 3:10 AM > To: Jiangning Liu > Cc: gcc-patches@gcc.gnu.org; r.

Go patch committed: Update library to r60.1 release

2011-09-21 Thread Ian Lance Taylor
This patch updates the Go library to the r60.1 release. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r dc1239c722f8 libgo/MERGE --- a/libgo/MERGE Wed Sep 21 21:00:54 2011 -0700 +++ b/libgo/MERGE Wed Sep 21 21:35:15 2011 -0700 @@ -1,4 +1,4 @@ -5

Re: [PATCH, ARM] Unaligned accesses for packed structures [1/2]

2011-09-21 Thread Ye Joey
Julian, This patch works for register ld/st, but doesn't work for immediate, as showed in example. Would you further improve it for imm? Thanks - Joey $ arm-none-eabi-gcc -v 2>&1 | grep version gcc version 4.7.0 20110922 (experimental) [trunk revision 179074] (GCC) $ cat u.c struct packed_str {

Re: [PATCH] Add VIS intrinsics header for sparc.

2011-09-21 Thread David Miller
From: Hans-Peter Nilsson Date: Wed, 21 Sep 2011 23:27:38 -0400 (EDT) > Minor inconsistency spotted there: the same goes for the fpack > insns but you now set TREE_READONLY for them. (Not claiming I > caught all of them.) Thanks a lot for pointing this out, I'll remove the TREE_READONLY flag for

[PATCH RFC] Add FMAF support on Sparc.

2011-09-21 Thread David Miller
Eric, could you give this a quick spin on Solaris? Even if you don't have a FMAF capable cpu available, at least make sure the configury assembler feature detection bits work properly. And any other form of feedback is appreciated as well. This patch works fine for me with current binutils unde

Re: [PR 47382] We cannot simply fold OBJ_TYPE_REF at all in 4.6

2011-09-21 Thread Maxim Kuvyrkov
On 9/02/2011, at 6:53 AM, Martin Jambor wrote: > > This patch basically disables all intraprocedural devirtualization > simply because that transformation relies on assumptions that no > longer hold true. That leaves only devirtualization within inlining > and IPA-CP but those do not work intrap

[arm-embedded] Backport mainline r178852, r172697

2011-09-21 Thread Joey Ye
Backport mainline r178852, r172697 to ARM/embedded-4_6-branch Committed. 2011-09-22 Joey Ye Backport r178852 from mainline 2011-09-14 Julian Brown * config/arm/arm.c (arm_override_options): Add unaligned_access support. (arm_file_start): Emit attrib