Re: [patch, doc] copy-edit inline asm sections

2015-02-06 Thread Jeff Law
On 02/05/15 10:13, Sandra Loosemore wrote: It looks to me like that, when the rewrite of the inline asm sections was committed last May (r210273), there was general agreement that it needed additional copy-editing to fix markup problems and the like. But, nobody ever followed up to do that.

Re: [PATCH]Keep location info when expand complex component-wise load/store.

2015-02-06 Thread Andrew Pinski
On Thu, Feb 5, 2015 at 3:22 AM, Renlin Li renlin...@arm.com wrote: Hi all, location information is missing when rewrite complex component-wise load/store. This patch add the location info to fix the recently gcc.dg/uninit-13.c and gcc.dg/uninit-17.c failures on arm target. The compiler

Re: [PATCH] Fix PR ipa/64858

2015-02-06 Thread Jeff Law
On 01/30/15 05:29, Martin Liška wrote: Hi. Attached patch handles situation where remove_unreachable_nodes is called before IPA ICF registers removal hooks. As a result, a deleted callgraph node is touched. Tested on x86_64-linux-pc without a new regression seen. Ready for trunk? Thanks,

[PATCH] Fix another case of modified global_options after creation of optimization_default_node

2015-02-06 Thread Jakub Jelinek
Hi! As discussed in PR64340, the clearing of flag_ipa_ra is misplaced too, because it isn't reflected in optimization_default_node. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2015-02-06 Jakub Jelinek ja...@redhat.com * toplev.c

[PATCH, i386 testsuite]: Fix gcc.target/i386/pr64317.c

2015-02-06 Thread Uros Bizjak
Hello! 2015-02-06 Uros Bizjak ubiz...@gmail.com * gcc.target/i386/pr64317.c: Compile for 32bit *-*-linux* targets. (dg-options): Use -fpie instead of -fPIE -pie. Tested on x86_64-linux-gnu {,m32}, committed to mainline SVN. Uros. Index: gcc.target/i386/pr64317.c

[PATCH] Update optimization_default_node in ada (PR middle-end/64340)

2015-02-06 Thread Jakub Jelinek
Hi! As mentioned in the PR, the problem here is that the Ada FE needs to modify global_options after toplevel.c (process_options), but as for LTO we now use optimization_{default,current}_node for options of functions without specifial optimization node, it means the changed options aren't

[Patch, fortran] PR63744 accept duplicate use-rename

2015-02-06 Thread Mikael Morin
Hello, we currently reject programs of the form module m integer :: s end module m subroutine s use m, only: x = s, x = s end subroutine s with an error stating that S is the name of the current program unit. Interestingly, the duplicate rename is necessary to trigger it. There

Re: [PATCH v2][MIPS] fix CRT_CALL_STATIC_FUNCTION macro

2015-02-06 Thread Maciej W. Rozycki
On Fri, 6 Feb 2015, Mike Stump wrote: Personally, the call form of bal in my book should be called call, and the non-call form of it should be called bal, but, I realize it is likely to late to do much about now. If one went down this path, then even changing it away from bal is wrong.

Re: [PATCH] Fix another case of modified global_options after creation of optimization_default_node

2015-02-06 Thread Jeff Law
On 02/06/15 13:28, Jakub Jelinek wrote: Hi! As discussed in PR64340, the clearing of flag_ipa_ra is misplaced too, because it isn't reflected in optimization_default_node. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2015-02-06 Jakub Jelinek

Re: [gofrontend-dev] [PATCH 1/2, libgo] Add support for missing reflection functions to gccgo for ppc64, ppc64le in gcc 4.9

2015-02-06 Thread Ian Lance Taylor
On Fri, Feb 6, 2015 at 12:36 PM, Lynn A. Boger labo...@linux.vnet.ibm.com wrote: I was told by the testers that this patch did not work. The error message was: panic: interface conversion: interface is nil, not error goroutine 1 [running]: client.$nested1

Re: [PATCH] PR rtl-optimization/32219: optimizer causes wrong code in pic/hidden/weak symbol checking

2015-02-06 Thread Jack Howarth
/src/fink.build/gcc50-5.0.0-1000/darwin_objdir/gcc/ /sw/src/fink.build/gcc50-5.0.0-1000/gcc-5-20150206/gcc/testsuite/gcc.dg/visibility-22.c -fno-diagnostics-show-caret -fdiagnostics-color=never -fPIC -lm -m32 -o ./visibility-22.exe(timeout = 300) spawn -ignore SIGHUP /sw/src/fink.build/gcc50

Re: [PATCH] PR rtl-optimization/32219: optimizer causes wrong code in pic/hidden/weak symbol checking

2015-02-06 Thread Jack Howarth
/gcc-5-20150206/gcc/testsuite/gcc.dg/visibility-23.c -fno-diagnostics-show-caret -fdiagnostics-color=never -fPIC -S -m32 -o visibility-23.s contains... .text .globl _main _main: LFB0: pushl %ebp LCFI0: movl%esp, %ebp LCFI1: subl$8, %esp call

[PATCH][PR rtl-optimization/42522] Incorrect simplification of ZERO_EXTRACT and SIGN_EXTRACT by cse

2015-02-06 Thread Jeff Law
This bug has gone latent on the trunk; however, the problem still remains that cse will incorrectly simplify a ZERO/SIGN_EXTRACT in some cases. ZERO/SIGN_EXTRACT are somewhat special in that if they are extracting from a memory operand, that memory operand will always have QImode regardless of

Re: [PATCH] PR rtl-optimization/32219: optimizer causes wrong code in pic/hidden/weak symbol checking

2015-02-06 Thread H.J. Lu
with... Executing on host: /sw/src/fink.build/gcc50-5.0.0-1000/darwin_objdir/gcc/xgcc -B/sw/src/fink.build/gcc50-5.0.0-1000/darwin_objdir/gcc/ /sw/src/fink.build/gcc50-5.0.0-1000/gcc-5-20150206/gcc/testsuite/gcc.dg/visibility-22.c -fno-diagnostics-show-caret -fdiagnostics-color=never -fPIC -lm -m32

[wwwdocs, committed] Add link to jit docs to onlinedocs/index.html

2015-02-06 Thread David Malcolm
The attached patch adds a link to the built HTML docs for libgccjit to htdocs/onlinedocs/index.html. Committed to CVS as obvious. Index: htdocs/onlinedocs/index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/onlinedocs/index.html,v

Re: emit-rtl tidy

2015-02-06 Thread David Edelsohn
* config/rs6000/rs6000.c (rs6000_call_aix): Use get_current_sequence. The rs6000 change is okay. Thanks, David

[PATCH, committed] PR jit/64752 - Rewrite jit testsuite to eliminate use of file

2015-02-06 Thread David Malcolm
The jit testsuite attempted to test the gcc_jit_context_compile_to_file API entrypoint by running file on the generated file, and comparing the result against a regexp. This approach is unfixable: some hosts won't have file installed, and the output of file seems to vary enough from host to host

Re: [patch] Fix invalid attributes in libstdc++

2015-02-06 Thread Jonathan Wakely
On 3 February 2015 at 13:49, Renlin Li renlin...@arm.com wrote: On 01/02/15 15:08, Jonathan Wakely wrote: I failed to CC gcc-patches on this patch ... On 29/01/15 13:02 +, Jonathan Wakely wrote: diff --git a/libstdc++-v3/testsuite/17_intro/headers/c++200x/all_attributes.cc

Re: [PATCH][libstdc++][Testsuite] isctype test fails for newlib.

2015-02-06 Thread Jonathan Wakely
Any idea why HP still sees the tests fail? See comment 8 at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64467#c8

Re: [PATCH/AARCH64] Fix 64893: ICE with vget_lane_u32 with C++ front-end at -O0

2015-02-06 Thread Andrew Pinski
On Mon, Feb 2, 2015 at 11:37 PM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Feb 02, 2015 at 02:51:43PM -0800, Andrew Pinski wrote: While trying to build the GCC 5 with GCC 5, I ran into an ICE when building libcpp at -O0. The problem is the C++ front-end was not folding

Re: [doc, committed] fix typos in docs/comments for devirtualization optimizations

2015-02-06 Thread Gerald Pfeifer
Hi Sandra, Index: gcc/ipa-devirt.c === polymorphic (indirect) call - This is callgraph represention of virtual method call. Every + This is callgraph representation of virtual method call. Every polymorphic

Re: [PATCH] PR rtl-optimization/32219: optimizer causes wrong code in pic/hidden/weak symbol checking

2015-02-06 Thread Jack Howarth
/src/fink.build/gcc50-5.0.0-1000/darwin_objdir/gcc/ /sw/src/fink.build/gcc50-5.0.0-1000/gcc-5-20150206/gcc/testsuite/gcc.dg/visibility-22.c -fno-diagnostics-show-caret -fdiagnostics-color=never -fPIC -lm -m32 -o ./visibility-22.exe(timeout = 300) spawn -ignore SIGHUP /sw/src/fink.build

Re: [PATCH] PR rtl-optimization/32219: optimizer causes wrong code in pic/hidden/weak symbol checking

2015-02-06 Thread H.J. Lu
/sw/src/fink.build/gcc50-5.0.0-1000/darwin_objdir/gcc/ /sw/src/fink.build/gcc50-5.0.0-1000/gcc-5-20150206/gcc/testsuite/gcc.dg/visibility-23.c -fno-diagnostics-show-caret -fdiagnostics-color=never -fPIC -S -m32 -o visibility-23.s Can you give my updated patch: https://gcc.gnu.org/git/?p=gcc.git

[patch, libgfortran] [4.8/4.9/5 Regression] error reading (and writing) large text files in gfortran

2015-02-06 Thread Jerry DeLisle
With the attached patch I create a special version of fbuf_flush that is only called with list directed I/O. There can be no tabbing back and forth so it is safe to flush the buffer whenever we want. The bug occurs when the buffer keeps growing to no end until no more allocations can be made

Re: [PATCH] PR rtl-optimization/32219: optimizer causes wrong code in pic/hidden/weak symbol checking

2015-02-06 Thread Jack Howarth
/fink.build/gcc50-5.0.0-1000/gcc-5-20150206/gcc/testsuite/gcc.dg/visibility-22.c -fno-diagnostics-show-caret -fdiagnostics-color=never -fPIC -lm -m32 -o ./visibility-22.exe(timeout = 300) spawn -ignore SIGHUP /sw/src/fink.build/gcc50-5.0.0-1000/darwin_objdir/gcc/xgcc -B/sw/src/fink.build

[PATCH] PR 64256

2015-02-06 Thread David Edelsohn
After a lot of investigation, I believe that I have learned why stabs debugging on AIX disabled use of continuations. GDB, IBM DBX and IBM XLDB are able to work with stab string continuations produced by GCC. I am enabling it using definitions that match the behavior of IBM XL compilers to avoid

Re: [PATCH] PR rtl-optimization/32219: optimizer causes wrong code in pic/hidden/weak symbol checking

2015-02-06 Thread H.J. Lu
private_extern[ \t_]*_?foo with... Executing on host: /sw/src/fink.build/gcc50-5.0.0-1000/darwin_objdir/gcc/xgcc -B/sw/src/fink.build/gcc50-5.0.0-1000/darwin_objdir/gcc/ /sw/src/fink.build/gcc50-5.0.0-1000/gcc-5-20150206/gcc/testsuite/gcc.dg/visibility-22.c -fno-diagnostics-show-caret -fdiagnostics

Re: [doc, committed] fix typos in docs/comments for devirtualization optimizations

2015-02-06 Thread Sandra Loosemore
On 02/06/2015 06:19 PM, Gerald Pfeifer wrote: Hi Sandra, Index: gcc/ipa-devirt.c === polymorphic (indirect) call - This is callgraph represention of virtual method call. Every + This is callgraph representation

patch fixing a typo in LRA

2015-02-06 Thread Vladimir Makarov
The following patch fixes a typo discovered recently: https://gcc.gnu.org/ml/gcc/2015-02/msg00040.html The patch was bootstrapped and tested on x86/x86-64. Committed as rev.220488. 2015-02-06 Vladimir Makarov vmaka...@redhat.com * lra.c (lra_emit_add): Fix a typo in using

[PATCH] Fix thunk expansion (PR ipa/64896)

2015-02-06 Thread Jakub Jelinek
Hi! As discussed in the PR, for functions that return an aggregate that is not aggregate_value_p (i.e. returned in registers), using RESULT_DECL is undesirable, that's not what we normally emit for user code. So, this patch instead uses a temporary, which is optimized right now as much as

Re: [PATCH] Fix thunk expansion (PR ipa/64896)

2015-02-06 Thread Jan Hubicka
Hi! As discussed in the PR, for functions that return an aggregate that is not aggregate_value_p (i.e. returned in registers), using RESULT_DECL is undesirable, that's not what we normally emit for user code. So, this patch instead uses a temporary, which is optimized right now as much

RE: [PATCH v2][MIPS] fix CRT_CALL_STATIC_FUNCTION macro

2015-02-06 Thread Matthew Fortune
Maciej W. Rozycki ma...@linux-mips.org writes: On Thu, 5 Feb 2015, Matthew Fortune wrote: I'm OK with this change but I'd like Catherine to comment before committing. It seems a shame to duplicate the block of code but it is probably just as ugly to define a macro for the la/dla

Re: [Patch, WWWDOCS] gcc-5/changes.html: Mention that C FE can be build as libcc1.so

2015-02-06 Thread James Greenhalgh
On Fri, Feb 06, 2015 at 09:44:20AM +, Tobias Burnus wrote: I think it is useful to know that one can build libcc1.so - also as advertisement. Thus, I propose to include something like the quip in the attachment. GDB uses it in version 7.9, which is not yet released but already branched

[committed] Fix recent simplify_binary_operation_1 bug (PR rtl-optimization/64957)

2015-02-06 Thread Jakub Jelinek
Hi! The /* Given (xor (ior (xor A B) C) D), where B, C and D are constants, simplify to (xor (ior A C) (B~C)^D), canceling out bits inverted twice and not set by C. Similarly, given (xor (and (xor A B) C) D), simplify without inverting C in the xor

Re: [Patch, WWWDOCS] gcc-5/changes.html: Mention that C FE can be build as libcc1.so

2015-02-06 Thread Jakub Jelinek
On Fri, Feb 06, 2015 at 10:44:20AM +0100, Tobias Burnus wrote: I think it is useful to know that one can build libcc1.so - also as advertisement. Thus, I propose to include something like the quip in the attachment. GDB uses it in version 7.9, which is not yet released but already branched

Re: [PATCH] Fix DECL_ABSTRACT_P/BLOCK_ABSTRACT handling in dwarf2out.c (PR middle-end/64937)

2015-02-06 Thread Richard Biener
On February 5, 2015 4:42:50 PM CET, Jakub Jelinek ja...@redhat.com wrote: Hi! On the following testcase we fail -fcompare-debug, because in .LASAN0 initializer we get a NOP_EXPR optimized away only in one of the cases. The reason is that cgraph* is looking at DECL_ABSTRACT_P flag of

[PATCH] libssp should not use /dev/random on Windows

2015-02-06 Thread Georg Koppen
Hi, inline is a patch to avoid using /dev/random on Windows in ssp.c. If it is getting used there might be a local malicious process supplying fake random values (e.g. via C:\dev\random) rendering SSP useless. Comments/review are much appreciated. The patch is against the 4.9 branch: From

[Patch, WWWDOCS] gcc-5/changes.html: Mention that C FE can be build as libcc1.so

2015-02-06 Thread Tobias Burnus
I think it is useful to know that one can build libcc1.so - also as advertisement. Thus, I propose to include something like the quip in the attachment. GDB uses it in version 7.9, which is not yet released but already branched off. OK? - Other suggestions or comments? Tobias Index:

RE: [PATCH v2][MIPS] fix CRT_CALL_STATIC_FUNCTION macro

2015-02-06 Thread Maciej W. Rozycki
On Thu, 5 Feb 2015, Matthew Fortune wrote: I'm OK with this change but I'd like Catherine to comment before committing. It seems a shame to duplicate the block of code but it is probably just as ugly to define a macro for the la/dla instruction. Native systems have sys/asm.h for such ABI

Re: [PATCH, fortran, committed] PR60289 was: PR fortran/60255 Deferred character length + PR60289 Also deferred char len.

2015-02-06 Thread Andre Vehreschild
Hi Paul, thanks for the review. Committed as r220474. Regards, Andre On Thu, 5 Feb 2015 15:15:02 +0100 Paul Richard Thomas paul.richard.tho...@gmail.com wrote: Dear Andre, That's fine to commit to trunk. Thanks for the patch Paul On 30 January 2015 at 11:19, Andre

Re: [PATCH] Fix DECL_ABSTRACT_P/BLOCK_ABSTRACT handling in dwarf2out.c (PR middle-end/64937)

2015-02-06 Thread Jakub Jelinek
On Fri, Feb 06, 2015 at 09:44:40AM +0100, Richard Biener wrote: Looks good to me. I wonder if this will also help Aldyh... I also wonder what it takes to make dwarf2out use DECL_ABSTRACT_P || force_abstract with maintaining that new state and whether that would be both cheaper and cleaner?

Re: [PATCH] maintainer-scripts: Fix 404s in Show Source in jit docs

2015-02-06 Thread Gerald Pfeifer
On Thursday 2015-02-05 22:49, David Malcolm wrote: Tested locally with a suitably hacked up script to work on this machine; with this patch the Show Sources links work within the files installed to /www/gcc/htdocs. OK for trunk? maintainer-scripts/ChangeLog: * update_web_docs_svn:

[PATCH][5 regression][PR target/64889/] Fix incorrect setting of RTX_FRAME_RELATED_P on H8/300 port

2015-02-06 Thread Jeff Law
The H8/300 port is setting RTX_FRAME_RELATED_P on random insns outside the prologue/epilogue. That is causing dwarf2cfi heartburn as seen by PR64889 (and its dup 64602). This patch avoids setting RTX_FRAME_RELATED_P on those random insns. The H8/300 port can now build libssp properly

[PATCH], PR target/64205, Fix -mcpu=G5 breakage on PowerPC

2015-02-06 Thread Michael Meissner
This patch fixes PR target/64205, which was caused due to my previous patch to add the -mupper-regs support for scalar floating point to occupy the VSX floating point registers that are overlaid on top of the traditional Altivec registers, starting with ISA 2.06 (power7). The problem was, I added

Re: [debug-early] C++ clones and limbo DIEs

2015-02-06 Thread Jason Merrill
On 02/05/2015 08:15 PM, Aldy Hernandez wrote: Absolutely nothing, basically because #if 0 would have been too obvious. It was actually there to test if you were on your game. You passed Yay? The attached patch still passes the guality tests with no regressions. My apologies for the lack

RE: [MIPS] fix CRT_CALL_STATIC_FUNCTION macro

2015-02-06 Thread Maciej W. Rozycki
On Thu, 5 Feb 2015, Petar Jovanovic wrote: This is a follow-up to a change [1] in glibc. It fixes the issue [2] when jal can not reach a target in different region. Is it not a problem that can be solved with rearranging the order of sections in output? Is not this more clean

Re: PR lto/64837: lto plugin doesn't call ld_plugin_release_input_file

2015-02-06 Thread H.J. Lu
On Fri, Feb 6, 2015 at 5:18 AM, Markus Trippelsdorf mar...@trippelsdorf.de wrote: On 2015.02.06 at 05:10 -0800, H.J. Lu wrote: On Fri, Feb 6, 2015 at 4:56 AM, Markus Trippelsdorf mar...@trippelsdorf.de wrote: On 2015.02.05 at 12:57 -0800, H.J. Lu wrote: We should pass handle, not file, to

RE: [PATCH v2][MIPS] fix CRT_CALL_STATIC_FUNCTION macro

2015-02-06 Thread Moore, Catherine
-Original Message- From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] Sent: Thursday, February 05, 2015 3:52 PM Subject: RE: [PATCH v2][MIPS] fix CRT_CALL_STATIC_FUNCTION macro I've put your patch inline below and switched to plain text. I suspect your post was bounced by

Re: [PATCH] Fix DECL_ABSTRACT_P/BLOCK_ABSTRACT handling in dwarf2out.c (PR middle-end/64937)

2015-02-06 Thread Aldy Hernandez
Richard Biener richard.guent...@gmail.com writes: Looks good to me. I wonder if this will also help Aldyh... Hmmm, I ran into something very similar. We solved it on the debug-early branch by avoiding recursively setting DECL_ABSTRACT_P if the parent was *not* DECL_ABSTRACT_P. Like this: @@

Re: [PATCH] doc/invoke.texi: clarify default setting of VTA

2015-02-06 Thread Alexander Monakov
Ping? On Fri, 30 Jan 2015, Alexander Monakov wrote: Hello, Recently on gcc-help@ one of the users asked whether they need to set -fvar-tracking-assignments on the command line by hand. The documentation may be clearer in that this flag has a useful value by default, like -fvar-tracking

[Haifa Scheduler] Fix latent bug in macro-fusion/instruction grouping

2015-02-06 Thread James Greenhalgh
Hi, While trying to force some particular conditions out of the scheduler on a private branch, I was toying with some very aggressive macro-fusion patterns. In doing so, I hit on a latent bug which can cause miscompilation. The exact conditions for the bug seem to be: We go in to macro fusion

Re: PR lto/64837: lto plugin doesn't call ld_plugin_release_input_file

2015-02-06 Thread Markus Trippelsdorf
On 2015.02.06 at 05:10 -0800, H.J. Lu wrote: On Fri, Feb 6, 2015 at 4:56 AM, Markus Trippelsdorf mar...@trippelsdorf.de wrote: On 2015.02.05 at 12:57 -0800, H.J. Lu wrote: We should pass handle, not file, to release_input_file. I checked it in as an obvious fix. This commit causes:

RE: [PATCH v2][MIPS] fix CRT_CALL_STATIC_FUNCTION macro

2015-02-06 Thread Maciej W. Rozycki
On Fri, 6 Feb 2015, Matthew Fortune wrote: Native systems have sys/asm.h for such ABI dependencies, including stuff to set up $gp. Perhaps we could reuse these bits, the licence I think allows us to. That's a good idea. Perhaps I should take that on as part of some cleanup of the MIPS

Re: PR lto/64837: lto plugin doesn't call ld_plugin_release_input_file

2015-02-06 Thread Markus Trippelsdorf
On 2015.02.05 at 12:57 -0800, H.J. Lu wrote: We should pass handle, not file, to release_input_file. I checked it in as an obvious fix. This commit causes: % echo int main () {} | gcc -fuse-ld=gold -flto -x c++ - ld.gold: internal error in remove_writer, at token.h:132 collect2: error: ld

Re: PR lto/64837: lto plugin doesn't call ld_plugin_release_input_file

2015-02-06 Thread H.J. Lu
On Fri, Feb 6, 2015 at 4:56 AM, Markus Trippelsdorf mar...@trippelsdorf.de wrote: On 2015.02.05 at 12:57 -0800, H.J. Lu wrote: We should pass handle, not file, to release_input_file. I checked it in as an obvious fix. This commit causes: % echo int main () {} | gcc -fuse-ld=gold -flto -x

Re: PR lto/64837: lto plugin doesn't call ld_plugin_release_input_file

2015-02-06 Thread H.J. Lu
On Fri, Feb 6, 2015 at 5:30 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, Feb 6, 2015 at 5:18 AM, Markus Trippelsdorf mar...@trippelsdorf.de wrote: On 2015.02.06 at 05:10 -0800, H.J. Lu wrote: On Fri, Feb 6, 2015 at 4:56 AM, Markus Trippelsdorf mar...@trippelsdorf.de wrote: On 2015.02.05

RE: [PATCH,WWWDOCS] MIPS changes for GCC 5.0

2015-02-06 Thread Moore, Catherine
-Original Message- From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] Sent: Thursday, February 05, 2015 5:24 AM To: Moore, Catherine Cc: 'gcc-patches@gcc.gnu.org' (gcc-patches@gcc.gnu.org) Subject: RE: [PATCH,WWWDOCS] MIPS changes for GCC 5.0 Moore, Catherine

Re: [PATCH v2][MIPS] fix CRT_CALL_STATIC_FUNCTION macro

2015-02-06 Thread Mike Stump
On Feb 6, 2015, at 4:23 AM, Maciej W. Rozycki ma...@linux-mips.org wrote: This consideration made me realise I've had a patch outstanding for some 10 years to convert all the `BAL x' instructions there to `BLTZAL $0, x'. This has always been a good idea in case implementations recognised the

RE: [PATCH v2][MIPS] fix CRT_CALL_STATIC_FUNCTION macro

2015-02-06 Thread Matthew Fortune
Mike Stump mikest...@comcast.net writes: On Feb 6, 2015, at 4:23 AM, Maciej W. Rozycki ma...@linux-mips.org wrote: This consideration made me realise I've had a patch outstanding for some 10 years to convert all the `BAL x' instructions there to `BLTZAL $0, x'. This has always been a

Re: [PATCH v2][MIPS] fix CRT_CALL_STATIC_FUNCTION macro

2015-02-06 Thread Maciej W. Rozycki
On Fri, 6 Feb 2015, Mike Stump wrote: This consideration made me realise I've had a patch outstanding for some 10 years to convert all the `BAL x' instructions there to `BLTZAL $0, x'. This has always been a good idea in case implementations recognised the special case and avoided

[PATCH][4.9]Backport Fix register corruption bug in ree

2015-02-06 Thread Renlin Li
Hi all, This is a backport patch for branch 4.9. You can find the original patch here: https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00356.html And it has been commit on the trunk as r215205. This fixes a few libstdc++-v3 test suite failures. x86_64 bootstraps Okay, aarch64_be-none-elf

Re: [debug-early] C++ clones and limbo DIEs

2015-02-06 Thread Jason Merrill
On 02/06/2015 11:42 AM, Aldy Hernandez wrote: I was actually thinking of using dwarf2out_early_finish() to mop things up as we generate early (or stream out) other auxiliary tables (pubname_table, pubtype_table, file_table, etc). More details on that later. If so, can I leave it as is?

Re: [PATCH], PR target/64205, Fix -mcpu=G5 breakage on PowerPC

2015-02-06 Thread David Edelsohn
On Fri, Feb 6, 2015 at 9:41 AM, Michael Meissner meiss...@linux.vnet.ibm.com wrote: This patch fixes PR target/64205, which was caused due to my previous patch to add the -mupper-regs support for scalar floating point to occupy the VSX floating point registers that are overlaid on top of the

Re: emit-rtl tidy

2015-02-06 Thread Richard Henderson
On 02/05/2015 04:18 PM, Alan Modra wrote: OK when stage1 opens? * function.h (struct emit_status): Delete x_first_insn, x_last_insn and sequence_stack. Add seq. (seq_stack): Delete. * function.c (prepare_function_start): Don't access x_last_insn. * emit-rtl.h

Re: [debug-early] C++ clones and limbo DIEs

2015-02-06 Thread Aldy Hernandez
OK with that change. Sweet! Thanks for everything. Though he's been silent, I bet Richi is secretly dancing :-).

[PATCH] PR rtl-optimization/32219: optimizer causes wrong code in pic/hidden/weak symbol checking

2015-02-06 Thread H.J. Lu
If a hidden weak symbol isn't defined in the TU, we can't assume it will be defined in another TU at link time. It makes a difference in code generation when compiling for PIC. If we assume that a hidden weak undefined symbol is local, the address checking may be optimized out and leads to the

Re: [PATCH x86_64] Optimize access to globals in -fpie -pie builds with copy relocations

2015-02-06 Thread H.J. Lu
On Thu, Feb 5, 2015 at 2:05 PM, Sriraman Tallam tmsri...@google.com wrote: On Thu, Feb 5, 2015 at 11:59 AM, Richard Henderson r...@redhat.com wrote: On 02/05/2015 11:01 AM, H.J. Lu wrote: Can you elaborate why it depends on COPY relocation? There is no COPY relocation on x86-64. Ho hum, we

Re: [debug-early] C++ clones and limbo DIEs

2015-02-06 Thread Aldy Hernandez
+ DECL_CONTEXT (snode-decl) + TREE_CODE (DECL_CONTEXT (snode-decl)) != FUNCTION_DECL) I think this should be !decl_function_context (snode-decl), in case there's a class or BLOCK between the symbol and its enclosing function. Done, also for the iteration through reachable functions.

Re: [libgo] Remove Solaris 11.1+ zone_net_addr_t treatment

2015-02-06 Thread Ian Lance Taylor
On Mon, Nov 3, 2014 at 8:59 AM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: The recent godump changes broke Solaris 11.1+ bootstrap in libgo: before, gen-sysinfo.so had type _zone_net_addr_t struct { zna_family uint16; zna_plen uint16; zna_addru struct { znau_addr6 _in6_addr; }; }

Re: [PATCH v2][MIPS] fix CRT_CALL_STATIC_FUNCTION macro

2015-02-06 Thread Mike Stump
On Feb 6, 2015, at 9:41 AM, Matthew Fortune matthew.fort...@imgtec.com wrote: Mike Stump mikest...@comcast.net writes: On Feb 6, 2015, at 4:23 AM, Maciej W. Rozycki ma...@linux-mips.org wrote: This consideration made me realise I've had a patch outstanding for some 10 years to convert all