Re: [PATCH, ifcvt] Allow CC mode if HAVE_cbranchcc4 (fix s390 build)

2014-11-06 Thread Richard Henderson
On 11/06/2014 05:10 PM, Ulrich Weigand wrote: >>> +/* For s390, CC REG is general_operand. But cstorecc4 >>> only >>> + handles CCZ1, which can not handle others like CCU. >>> */ >>> + || GET_MODE_CLASS (GET_MODE (XEXP (cond, 0))) == MODE_CC); >>> >>

Re: [gofrontend-dev] Re: [PATCH 00/13] Go closures, libffi, and the static chain

2014-11-06 Thread Richard Henderson
On 11/06/2014 06:45 PM, Ian Taylor wrote: > On Thu, Nov 6, 2014 at 5:04 AM, Richard Henderson wrote: >> >> That said, this *may* not actually be a problem. It's not the direct >> (possibly >> lazy bound) call into libffi that needs a static chain, it's the indirect >> call >> that libffi produc

[gimple-classes, committed 08/17] tree-sra.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-sra.c (build_accesses_from_assign): Strengthen param "stmt" from gimple to gassign *. (scan_function): Add checked cast within case GIMPLE_ASSIGN. (sra_modify_constructor_assign): Strengthen param "stmt" from gimple to gas

[gimple-classes, committed 16/17] tree-ssa-pre.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-ssa-pre.c (compute_avail): Within case GIMPLE_ASSIGN, introduce local "assign_stmt" via an as_a and use it in place of "stmt" for typesafety. (eliminate_dom_walker::before_dom_children): Introduce local gassign * "assign_s

[gimple-classes, committed 10/17] tree-ssa-ifcombine.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-ssa-ifcombine.c (get_name_for_bit_test): Replace is_gimple_assign with a dyn_cast, strengthening local "def_stmt" from gimple to gassign *. (recognize_single_bit_test): Replace various is_gimple_assign with dyn_cast, intro

[gimple-classes, committed 17/17] tree-ssa-propagate.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-ssa-propagate.c (substitute_and_fold_dom_walker::before_dom_children): Add checked cast. Replace is_gimple_assign with a dyn_cast, introducing local "assign_stmt" and using it in place of "stmt" for typesafety. (may_propa

[gimple-classes, committed 06/17] tree-predcom.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-predcom.c (name_for_ref): Replace is_gimple_assign with a dyn_cast, introducing local gassign * "ref_assign", using it place of ref->stmt for typesafety. (find_looparound_phi): Likewise. Strengthen local "init_stmt" from

[gimple-classes, committed 11/17] tree-ssa-live.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-ssa-live.c (remove_unused_locals): Add a checked cast in region guarded by gimple_clobber_p. --- gcc/ChangeLog.gimple-classes | 5 + gcc/tree-ssa-live.c | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/gcc/Chang

Re: [PATCH 0/3] Minor tweaks to jit

2014-11-06 Thread David Malcolm
On Thu, 2014-11-06 at 16:51 -0700, Jeff Law wrote: > On 11/05/14 13:54, David Malcolm wrote: > > Here are some minor tweaks I've made on the JIT branch since v3 of > > the review began. > > > > Are the following OK for trunk, assuming the rest of the JIT work > > is approved? > > > > David Malcolm

[committed] Add PA trap insn

2014-11-06 Thread John David Anglin
The attached change implements a trap insn on PA. This insn is need to implement __builtin_trap() and to successfully build glibc 2.20. Tested on hppa-unknown-linux-gnu and hppa2.0w-hp-hpux11.11. Committed to trunk, 4.9 and 4.8. Dave -- John David Anglin dave.ang...@bell.net 2014-

[gimple-classes, committed 07/17] tree-scalar-evolution.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-scalar-evolution.c (follow_ssa_edge_in_rhs): Strengthen param "stmt" from gimple to gassign *. (follow_ssa_edge): Add checked cast within case GIMPLE_ASSIGN. (interpret_rhs_expr): Replace is_gimple_assign with a dyn_cast,

[gimple-classes, committed 05/17] tree-loop-distribution.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-loop-distribution.c (generate_memset_builtin): Strengthen locals "stmt" and "fn_call" from gimple to gassign * and gcall * respectively. (classify_partition): Strengthen locals "stmt", "store", "load" from gimple to gassig

[gimple-classes, committed 14/17] tree-ssa-phiopt.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-ssa-phiopt.c (jump_function_from_stmt): Strengthen param "stmt" from gimple to gassign *. (rhs_is_fed_for_value_replacement): Replace is_gimple_assign with a dyn_cast, strengthening local "def1" from gimple to gassign *. (

[gimple-classes, committed 13/17] tree-ssa-loop-*: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-ssa-loop-ch.c (pass_ch::execute): Add checked cast. * tree-ssa-loop-im.c (movement_possibility): Add checked casts. (stmt_cost): Likewise. (simple_mem_ref_in_stmt): Capture result of gimple_assign_single_p as a new local g

[gimple-classes, committed 04/17] tree-into-ssa.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-into-ssa.c (create_new_def_for): Add a checked cast. --- gcc/ChangeLog.gimple-classes | 4 gcc/tree-into-ssa.c | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/gcc/ChangeLog.gimple-classes b/gcc/ChangeLog.gimple-classe

[gimple-classes, committed 01/17] tree-vrp.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-vrp.c (stmt_overflow_infinity): Replace is_gimple_assign with a dyn_cast, introducing local "assign_stmt" and using it in place of "stmt" for typesafety. (gimple_assign_nonnegative_warnv_p): Strengthen param "stmt" from gi

[gimple-classes, committed 12/17] tree-ssa-math-opts.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-ssa-math-opts.c (is_division_by): Replace is_gimple_assign with dyn_cast, introducing local gassign * "use_assign", using it in place of "use_stmt" for typesafety. (replace_reciprocal): Add a checked cast. (pass_cse_recipr

[gimple-classes, committed 15/17] tree-ssa-phiprop.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-ssa-phiprop.c (phiprop_insert_phi): Strengthen param "use_stmt" from gimple to gassign *. Add checked cast. (propagate_with_phi): Add checked casts. Replace is_gimple_assign with a dyn_cast, introducing local "use_assign" and us

[gimple-classes, committed 09/17] tree-switch-conversion.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-switch-conversion.c (struct switch_conv_info): Strengthen fields "arr_ref_first" and "arr_ref_last" from gimple to gassign *. --- gcc/ChangeLog.gimple-classes | 6 ++ gcc/tree-switch-conversion.c | 4 ++-- 2 files changed, 8 insertio

[gimple-classes, committed 00/17] Use gassign * in 17 more files

2014-11-06 Thread David Malcolm
I've pushed the following 17 patches to the git branch "dmalcolm/gimple-classes". This is part of ongoing work on the branch to make all gimple_assign_* accessors take a gassign *, rather than a gimple. Successfully bootstrapped®rtested the combination of the 17 patches upon the branch on x86_64-

[gimple-classes, committed 02/17] tree-complex.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-complex.c (init_dont_simulate_again): Within case GIMPLE_ASSIGN, introduce local gassign *"assign_stmt" and use it in place of "stmt". (complex_visit_stmt): Add checked casts to gassign *. (expand_complex_move): Replace is

[gimple-classes, committed 03/17] tree-inline.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-inline.c (remap_gimple_stmt): Introduce local "assign_stmt" in region guarded by gimple_clobber_p (stmt) and use it in place of "stmt" for typesafety. (copy_bb): Replace is_gimple_assign with a dyn_cast, introducing local

Re: [PATCH 0/3] Minor tweaks to jit

2014-11-06 Thread Jeff Law
On 11/05/14 13:54, David Malcolm wrote: Here are some minor tweaks I've made on the JIT branch since v3 of the review began. Are the following OK for trunk, assuming the rest of the JIT work is approved? David Malcolm (3): New test cases Documentation tweak Add comments to various func

Re: Extract and insert merging patch

2014-11-06 Thread Jeff Law
On 11/05/14 13:46, Petr Murzin wrote: Hi, Bootstrapped. No regressions detected. Please have a look. Is it ok for trunk? 2014-11-05 Petr Murzin gcc/ * simplify-rtx.c (simplify_ternary_operation): Simplify vec_merge (vec_duplicate (vec_select)). gcc/testsuite/

Re: [PATCH v3] warning about const multidimensional array as function parameter

2014-11-06 Thread Joseph Myers
On Thu, 6 Nov 2014, Martin Uecker wrote: > This patch implements a new proposed behaviour for > pointers to arrays with qualifiers in C. > > I found some time to work on this again, so here is another > revision. Main changes to the previous version of the patch: > > - add more test cases for

Re: [PATCH] PR 63721 IPA ICF cause atomic-comp-swap-release-acquire.c ICE

2014-11-06 Thread Jeff Law
On 11/05/14 07:09, Jiong Wang wrote: the same ICE will happen on x86-64, if compile with -O2 -fPIC. the reason is for the following two functions, they are identical, so IPA-ICF pass try to transform the second function to call the first one directly. int atomic_compare_exchange_STRONG_RELEASE_

Re: [PATCH RFC]Pair load store instructions using a generic scheduling fusion pass

2014-11-06 Thread Jeff Law
On 11/05/14 02:30, Bin.Cheng wrote: [ ... ] rfs_result's signature has changed, I think you need to pass in tmp & tmp2. You'll need to make that trivial update for all the callers. Can you make those changes and repost so that I can look at the docs (I think the implementation is fine and won'

[PATCH][Revised] Fix PR63750

2014-11-06 Thread Jack Howarth
The revised attached patch, which corrects the placement of the include for below that of config.h, eliminates the compilation error... error: use of undeclared identifier 'do_not_use_toupper_with_safe_ctype' on x86_64-apple-darwin14 when bootstrapping using the Clang 6.0 compiler. Okay

[patch] OpenACC fortran tests

2014-11-06 Thread Cesar Philippidis
This patch contains compile-time tests for OpenACC in gfortran. Is this patch OK for mainline trunk after the OpenACC fortran front end changes make their way in? The fortran front end changes can be found here: https://gcc.gnu.org/ml/gcc-patches/2014-11/msg00553.html Thanks, Cesar 2014-11-06 C

[patch] OpenACC fortran front end

2014-11-06 Thread Cesar Philippidis
This patch adds support for OpenACC 2.0a, with some omissions, to the fortran front end. It only contains the fortran changes from gomp-4_0-branch, therefore the middle end and runtime changes are a necessary prerequisite for this patch. I'll post a separate patch with the fortran tests later. If a

[PATCH] Don't necessarily emit object size checks for ARRAY_REFs

2014-11-06 Thread Marek Polacek
First part of this patch is about removing the useless check that we talked about earlier today. The rest is about not emitting UBSAN_OBJECT_SIZE checks (those often come with multiple statements to compute a pointer difference) for ARRAY_REFs that are already instrumented by UBSAN_BOUNDS. I do t

Re: Update c99status.html for extended identifiers changes

2014-11-06 Thread Joseph Myers
On Thu, 6 Nov 2014, Marek Polacek wrote: > On Thu, Nov 06, 2014 at 09:31:46PM +, Joseph Myers wrote: > > I've applied this patch to update c99status.html for the recent changes > > regarding extended identifiers. The remaining known C99/C11 conformance > > issues (beyond ordinary bugs) are

Re: Update c99status.html for extended identifiers changes

2014-11-06 Thread Marek Polacek
On Thu, Nov 06, 2014 at 09:31:46PM +, Joseph Myers wrote: > I've applied this patch to update c99status.html for the recent changes > regarding extended identifiers. The remaining known C99/C11 conformance > issues (beyond ordinary bugs) are all floating-point issues (mostly but > not entir

Update c99status.html for extended identifiers changes

2014-11-06 Thread Joseph Myers
I've applied this patch to update c99status.html for the recent changes regarding extended identifiers. The remaining known C99/C11 conformance issues (beyond ordinary bugs) are all floating-point issues (mostly but not entirely Annex F/G). Index: c99status.html ===

Preserve original spellings of extended identifiers

2014-11-06 Thread Joseph Myers
This patch makes cpplib track the original spellings of extended identifiers, as well as the canonical UTF-8 version, in order to follow standard semantics properly without needing a convoluted and undocumented canonicalization in translation phase 1 (see bug 9449 comments 39-46 regarding such a ca

Re: [PATCH\ Fix PR63750

2014-11-06 Thread Joseph Myers
On Thu, 6 Nov 2014, Jack Howarth wrote: > 2014-10-06 Jack Howarth > > PR other/63750 > * gcc/ipa-chkp.c: Include sstream. > > Index: gcc/ipa-chkp.c > === > --- gcc/ipa-chkp.c(revision 217192) > +++ gcc/ipa-chkp.c

[ping] libatomic: Fix sub-word CAS synthesis on LP64 targets

2014-11-06 Thread Andrew Waterman
Greetings. This is just a ping about a one-liner. The original email was here and is inlined below. https://gcc.gnu.org/ml/gcc-patches/2014-10/msg02465.html A missing cast causes the compare value to be truncated, resulting in spurious CAS failures. The problem manifested in a new port I'm wor

Re: [gomp4] Move libgomp plugins into subdirectory

2014-11-06 Thread Julian Brown
On Thu, 6 Nov 2014 15:37:42 + Julian Brown wrote: > On Thu, 6 Nov 2014 11:11:42 +0100 > Jakub Jelinek wrote: > > > On Thu, Nov 06, 2014 at 10:06:00AM +0100, Thomas Schwinge wrote: > > > Hmm. And then we "cross-include" files in libgomp/ from > > > libgomp/plugin/ as well as the other way r

Re: [PATCH] PR36312

2014-11-06 Thread Anthony Brandon
Sorry, I didn't realize the default didn't build all languages. I will configure with --enable-languages=all from now on. On Thu, Nov 6, 2014 at 1:26 AM, Manuel López-Ibáñez wrote: > On 5 November 2014 21:57, Eric Botcazou wrote: >>> 2014-10-25 Anthony Brandon >>> >>> PR driver/36312 >>>

[patch sdbout]: Fix ICE on -debug testsuite test const2.C for coff

2014-11-06 Thread Kai Tietz
Hi, This fixes recent fallout of debug-tests on Windows target for sdbout (coff) caused by an ICE. ChangeLog 2014-11-06 Kai Tietz * sdbout.c (sdbout_symbol): Eliminate register only if decl isn't a global variable. Tested for i686-w64-mingw32, and x86_64-w64-mingw32. Ok for apply?

[PATCH\ Fix PR63750

2014-11-06 Thread Jack Howarth
The attached patch eliminates the compilation error... error: use of undeclared identifier 'do_not_use_toupper_with_safe_ctype' on x86_64-apple-darwin14 when bootstrapping using the Clang 6.0 compiler. Okay for gcc trunk? Jack PR63750.patch Description: Binary data

[PATCH] fix PR63699

2014-11-06 Thread Jack Howarth
The attached patch eliminates the compilation error... error: use of undeclared identifier 'do_not_use_toupper_with_safe_ctype' on x86_64-apple-darwin14 when bootstrapping using the Clang 6.0 compiler. Okay for gcc trunk? Jack PR63699.patch Description: Binary data

Re: [debug-early] emit locals early patchset

2014-11-06 Thread Richard Biener
On Wed, Nov 5, 2014 at 5:06 PM, Martin Jambor wrote: > On Tue, Oct 28, 2014 at 03:57:43PM +0100, Richard Biener wrote: >> On Tue, Oct 28, 2014 at 1:00 AM, Aldy Hernandez wrote: >> > Gentlemen! >> > >> > My apologies for the big patch. In getting locals emitted early >> > (parameters >> > and lo

RE: [Patch] MIPS configuration patch to enable --with-[arch,endian,abi]

2014-11-06 Thread Steve Ellcey
On Fri, 2014-10-31 at 15:12 -0700, Matthew Fortune wrote: > I'm not keen on adding support for --with-endian to mips. I believe that > various build systems have become quite dependent on 'mips' meaning big > endian and 'mipsel' being little endian. I'd prefer to remove that part > unless you (or

Re: [patch] Provide a can_compare_and_swap_p target hook.

2014-11-06 Thread Andrew MacLeod
On 11/06/2014 01:23 PM, Andrew Haley wrote: On 11/06/2014 05:57 PM, Andrew MacLeod wrote: It looks like java is deciding whether or not GCC can inline atomic operations or not, and if it can't, doesn't want the atomic operations... which presumably means there is no dependency on libatomic at

Re: [PATCH 10/11][RS6000] Migrate reduction optabs to reduc_..._scal

2014-11-06 Thread Bill Schmidt
On Thu, 2014-11-06 at 16:44 +, Alan Lawrence wrote: > Hmmm. I am a little surprised by your mention of "saturation points" as I > would > not expect any variety of reduc_plus to be a saturating operation??? I wouldn't either, but the underlying vsum4ubs and vsum4sbs instructions used in thes

Re: [PATCH 3/4] Add libgomp plugin for Intel MIC

2014-11-06 Thread Jakub Jelinek
On Mon, Oct 27, 2014 at 03:15:56PM +0300, Ilya Verbin wrote: > + bool use_alloca; > + const char *ld_lib_path = getenv (LD_LIBRARY_PATH_ENV); > + const char *mic_lib_path = getenv (MIC_LD_LIBRARY_PATH_ENV); > + char *mic_lib_path_new; > + size_t len; > + > + if (!ld_lib_path) > +return;

Re: [patch] Provide a can_compare_and_swap_p target hook.

2014-11-06 Thread Andrew Haley
On 11/06/2014 05:57 PM, Andrew MacLeod wrote: > It looks like java is deciding whether or not GCC can inline atomic > operations or not, and if it can't, doesn't want the atomic > operations... which presumably means there is no dependency on > libatomic at runtime. > > A call to can_compare_

Re: [PATCH 2/4] Add liboffloadmic

2014-11-06 Thread Jakub Jelinek
On Wed, Oct 22, 2014 at 11:21:28PM +0400, Ilya Verbin wrote: > > Also, do we really want the messy DOS/Windows '\r' in the messages on > > Unix-ish targets? Shouldn't that be dependent on what target is the library > > configured for? > > Fixed. ... I'm still seeing various unhandled malloc fai

[PATCH v3] warning about const multidimensional array as function parameter

2014-11-06 Thread Martin Uecker
This patch implements a new proposed behaviour for pointers to arrays with qualifiers in C. I found some time to work on this again, so here is another revision. Main changes to the previous version of the patch: - add more test cases for cast from/to (const) void* - correctly uses pedwarn/wa

Re: [PATCH]Partially fix PR61529, bound basic block frequency

2014-11-06 Thread Renlin Li
On 06/11/14 17:59, Teresa Johnson wrote: Thanks for fixing the test case. Can you also add the comment I suggested to the source change? Please add a comment that this is needed due to insane incoming frequencies. Sorry, I mistakenly add it to the ChangeLog. Should be correct now. x86_64-u

Re: [PATCH 1/4] Add mkoffload for Intel MIC

2014-11-06 Thread Jakub Jelinek
On Wed, Oct 22, 2014 at 10:57:01PM +0400, Ilya Verbin wrote: > gcc/ > * config.gcc (*-intelmic-* | *-intelmicemul-*): Add i386/t-intelmic to > tmake_file. > (i[34567]86-*-* | x86_64-*-*): Build mkoffload$(exeext) with the > accelerator compiler. > * config/i386/intelmi

Re: [PATCH]Partially fix PR61529, bound basic block frequency

2014-11-06 Thread Teresa Johnson
Thanks for fixing the test case. Can you also add the comment I suggested to the source change? > Please add a comment that this is needed due to insane incoming frequencies. Thanks, Teresa On Thu, Nov 6, 2014 at 9:53 AM, Renlin Li wrote: > Hi Teresa, > > Thank you for the suggestion, updated!

Re: [PATCH PR 63758] fix liblto_plugin.so undefined _environ reference on OSX host

2014-11-06 Thread Mike Stump
On Nov 5, 2014, at 4:25 PM, Roland McGrath wrote: > As I described in the bug, OSX (as documented) does not permit shared > libraries to refer directly to the environ global variable. > OK for trunk and 4.9 branch? For the darwin bits I can approve, Ok.

[m32c] tweaks to EH, cond, etc

2014-11-06 Thread DJ Delorie
Fixes newlib/libgcc build problems, many test cases. No regressions. Applied. * config/m32c/cond.md (movqicc__): Remove mode of conditional. (movhicc__): Likewise. * config/m32c/m32c.c (encode_pattern_1): Specialise PSImode subregs. (m32c_eh_return

Re: [patch] Provide a can_compare_and_swap_p target hook.

2014-11-06 Thread Andrew MacLeod
On 11/04/2014 03:13 PM, Andrew MacLeod wrote: >from targhooks.c: bool default_builtin_support_vector_misalignment (machine_mode mode, const_tree type, <...>) { if (optab_handler (movmisalign_optab, mode) != CODE_FOR_nothing) return true; return false; } the idea is to move all the

Re: [PATCH 4/4] OpenMP 4.0 offloading to Intel MIC: non-fallback testing

2014-11-06 Thread Jakub Jelinek
On Thu, Oct 30, 2014 at 02:40:01PM +0300, Ilya Verbin wrote: > 2014-10-30 Andrey Turetskiy > Ilya Verbin > > libgomp/ > * Makefile.in: Regenerate. > * configure: Regenerate. > * configure.ac: Set up offload_additional_options, > offload_additional_lib_paths a

Re: [PATCH]Partially fix PR61529, bound basic block frequency

2014-11-06 Thread Renlin Li
Hi Teresa, Thank you for the suggestion, updated! Please add a comment that this is needed due to insane incoming frequencies. The 'b' variable is uninitialized. Also, 'd' and 'a' may end up uninitialized depending on the initial value of 'b'. Please initialize these. Test case has been add

Re: [gofrontend-dev] Re: [PATCH 00/13] Go closures, libffi, and the static chain

2014-11-06 Thread Ian Taylor
On Thu, Nov 6, 2014 at 5:04 AM, Richard Henderson wrote: > > That said, this *may* not actually be a problem. It's not the direct > (possibly > lazy bound) call into libffi that needs a static chain, it's the indirect call > that libffi produces. And the indirect calls that Go produces. > > I'm

Re: [PATCH][AArch64] Add bounds checking to vqdm*_lane intrinsics via a qualifier that also flips endianness

2014-11-06 Thread Mike Stump
On Nov 6, 2014, at 2:19 AM, Alan Lawrence wrote: > + vqdmlal_high_lane_s16 (int32x4_a, int16x8_b, int16x4_c, 4); > +} > \ No newline at end of file Please ensure all new files end with a newline.

Re: [PATCH] Fix for mklog

2014-11-06 Thread Diego Novillo
On 11/06/14 03:00, Marat Zakirov wrote: Ok to commit? OK. Thanks. Diego.

RE: [PATCHv4][MIPS] Implement O32 ABI extensions (GCC)

2014-11-06 Thread Matthew Fortune
Sorry to follow myself up. I realised that the new configure options should be documented in install.texi. The only change from V3 is the doc/install.texi change. The MIPS64 tests have completed without regression. Regards, Matthew gcc/ * common/config/mips/mips-common.c (mips_handle_opt

Re: [gofrontend-dev] [PATCH 4/4] Gccgo port to s390[x] -- part II

2014-11-06 Thread Ian Taylor
On Thu, Nov 6, 2014 at 4:04 AM, Dominik Vogt wrote: > On Tue, Nov 04, 2014 at 08:16:51PM -0800, Ian Taylor wrote: >> The way to do it is not by >> copying the test. If the test needs to be customized, add additional >> files that use // +build lines to pick which files is built. Move >> them int

Re: ping x 7: [PATCH] [libgomp] make it possible to use OMP on both sides of a fork

2014-11-06 Thread Nathaniel Smith
Ping^2. On Tue, Oct 28, 2014 at 6:17 PM, Nathaniel Smith wrote: > Ping. > > On 19 Oct 2014 23:44, "Nathaniel Smith" wrote: >> >> Hi Jakub, >> >> Thanks for your feedback! See below. >> >> On Thu, Oct 16, 2014 at 4:52 PM, Jakub Jelinek wrote: >> > On Mon, Oct 13, 2014 at 10:16:19PM +0100, Nathan

Re: [gofrontend-dev] [PATCH 1/4] Gccgo port to s390[x] -- part II

2014-11-06 Thread Ian Taylor
On Wed, Nov 5, 2014 at 4:55 AM, Dominik Vogt wrote: > On Wed, Nov 05, 2014 at 11:31:28AM +0100, Dominik Vogt wrote: >> On Tue, Nov 04, 2014 at 02:39:34PM -0800, Ian Taylor wrote: >> > Note that libgo/runtime/runtime.c now refers to S390_HAVE_STCKF. It's >> > not obvious to me that that is defined

Re: [gofrontend-dev] [PATCH 1/4] Gccgo port to s390[x] -- part II

2014-11-06 Thread Ian Taylor
On Thu, Nov 6, 2014 at 7:34 AM, Dominik Vogt wrote: > On Thu, Nov 06, 2014 at 03:02:13PM +0100, Rainer Orth wrote: >> Ian Taylor writes: >> > Committed patch 0001 with various formatting fixes, as attached. >> >> Unfortunately, the mksysinfo.sh part broke Solaris bootstrap: there's no >> type _uc

Re: [PATCH 10/11][RS6000] Migrate reduction optabs to reduc_..._scal

2014-11-06 Thread Alan Lawrence
Hmmm. I am a little surprised by your mention of "saturation points" as I would not expect any variety of reduc_plus to be a saturating operation??? A. Bill Schmidt wrote: On Fri, 2014-10-24 at 19:49 -0400, David Edelsohn wrote: On Fri, Oct 24, 2014 at 8:06 AM, Alan Lawrence wrote: This mig

Re: [PATCH, ifcvt] Allow CC mode if HAVE_cbranchcc4 (fix s390 build)

2014-11-06 Thread Ulrich Weigand
Richard Henderson wrote: > On 11/06/2014 08:44 AM, Zhenqiang Chen wrote: > > Hi, > > > > The patch add runtime check to fix s390 build fail > > (https://gcc.gnu.org/ml/gcc-patches/2014-11/msg00050.html). > > > > And there is additional code to workaround s390 cstorecc4 issue. > > > > Bootstrap a

Re: [PATCH]Partially fix PR61529, bound basic block frequency

2014-11-06 Thread Teresa Johnson
On Thu, Nov 6, 2014 at 7:09 AM, Renlin Li wrote: > > Hi Jeff, > > Test case has been added. With the patch, both x86_64-unknown-linux-gnu and > aarch64-none-elf compile the test case successfully. > > Okay to commit? > > > On 04/11/14 21:59, Jeff Law wrote: >> >> On 11/03/14 08:29, Renlin Li wrote

Re: [gomp4] Move libgomp plugins into subdirectory

2014-11-06 Thread Julian Brown
On Thu, 6 Nov 2014 11:11:42 +0100 Jakub Jelinek wrote: > On Thu, Nov 06, 2014 at 10:06:00AM +0100, Thomas Schwinge wrote: > > Hi Julian! > > > > On Wed, 5 Nov 2014 17:57:10 +, Julian Brown > > wrote: > > > This patch moves plugin-nvptx.c and plugin-host.c (from > > > oacc-host.c) into a new

[PATCH] RTEMS: select SPARC multilibs

2014-11-06 Thread Daniel Hellstrom
Recent support for mcpu=leon3v7 and muser-mode were added to GCC. Update the RTEMS multilib for sparc to the following combinations: v7- ./ leon3 muser-mode - leon3/user-mode/ leon3v7 muser-mode- leon3v7/user-mode/ v8

[gimple-classes, committed 17/44] tree-affine.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-affine.c (aff_combination_expand): Replace is_gimple_assign with a dyn_cast, strengthening local "def" from gimple to gassign *. --- gcc/ChangeLog.gimple-classes | 6 ++ gcc/tree-affine.c| 5 ++--- 2 files changed, 8 inse

Re: [gofrontend-dev] [PATCH 1/4] Gccgo port to s390[x] -- part II

2014-11-06 Thread Dominik Vogt
On Thu, Nov 06, 2014 at 03:02:13PM +0100, Rainer Orth wrote: > Ian Taylor writes: > > Committed patch 0001 with various formatting fixes, as attached. > > Unfortunately, the mksysinfo.sh part broke Solaris bootstrap: there's no > type _ucred in gen-sysinfo.go, so the grep in upcase_fields fails.

[gomp4] OpenACC Executable Directives

2014-11-06 Thread Thomas Schwinge
Hi Jim! On Tue, 4 Nov 2014 14:18:43 -0600, James Norris wrote: > void > f (int flag,, int *a) > { > if (flag) > #pragma acc update host (a[0:10]) > > return; > } > > I read bullet one under "Restrictions" on page 43, to mean the > placement of the update directive statement in

Re: [PATCH]Partially fix PR61529, bound basic block frequency

2014-11-06 Thread Renlin Li
Hi Jeff, Test case has been added. With the patch, both x86_64-unknown-linux-gnu and aarch64-none-elf compile the test case successfully. Okay to commit? On 04/11/14 21:59, Jeff Law wrote: On 11/03/14 08:29, Renlin Li wrote: On 29/10/14 12:42, Teresa Johnson wrote: Hi Renlin, Are the inc

[PING][PATCH][AARCH64]Fix PR63424 by adding v2di3 pattern

2014-11-06 Thread Renlin Li
Hi all, Dose anybody have time to review this? Kind regards, Renlin Li On 31/10/14 14:51, Renlin Li wrote: Hi all, This is a patch which will fix PR63424. It implements signed/unsigned max/min pattern for V2DI mode in terms of vcondv2div2di pattern. In this particular case, VEC_COND_EXPR

Re: [patch] Fix ACATS c65003a

2014-11-06 Thread Eric Botcazou
> Can you instead of initializing it with 0, turn it into a default > definition? > > Thus do sth like > > SSA_NAME_VAR (lhs) = create_tmp_reg (TREE_TYPE (lhs), NULL); > set_ssa_default_def (cfun, SSA_NAME_VAR (lhs), lhs); This occurred to me, but I didn't feel comfortable about fiddling with th

[gimple-classes, committed 13/44] rewrite_to_defined_overflow takes a gassign *

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * gimple-fold.h (rewrite_to_defined_overflow): Strengthen from gimple to gassign *. * gimple-fold.c (rewrite_to_defined_overflow): Likewise. * tree-scalar-evolution.c (scev_const_prop): Replace is_gimple_assign with a dyn_cast, i

[gimple-classes, committed 20/44] tree-if-conv.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-if-conv.c (parse_predicate): Replace is_gimple_assign with a dyn_cast, strengthening local "s" from gimple to gassign *. (if_convertible_gimple_assign_stmt_p): Strengthen param "stmt" from gimple to gassign *. (if_converti

[gimple-classes, committed 16/44] tree-object-size.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-object-size.c (plus_stmt_object_size): Strengthen param "stmt" from gimple to gassign *. (cond_expr_object_size): Likewise. (collect_object_sizes_for): Within case GIMPLE_ASSIGN, introduce local gassign * "assign_stmt" and

[gimple-classes, committed 22/44] tree-nested: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-nested.c (convert_local_reference_stmt): Add a checked cast within case GIMPLE_ASSIGN. --- gcc/ChangeLog.gimple-classes | 5 + gcc/tree-nested.c| 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/gcc/ChangeLog.g

[gimple-classes, committed 19/44] tree-data-ref.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-data-ref.c (split_constant_offset_1): Replace check against GIMPLE_ASSIGN with a dyn_cast, strengthening local "def_stmt" from gimple to gassign *. (get_references_in_stmt): Likewise, introducing local "assign_stmt" and us

[gimple-classes, committed 33/44] Use gassign within gimple_expr_type

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * gimple.h (gimple_expr_type): Split out if clause handling GIMPLE_ASSIGN and GIMPLE_CALL with an inner if GIMPLE_CALL into a pair of if clauses, first for GIMPLE_CALL, then for GIMPLE_ASSIGN. Replace them with dyn_casts, introducing lo

[gimple-classes, committed 18/44] tree-call-cdce.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-call-cdce.c (check_pow): Replace check against GIMPLE_ASSIGN with a dyn_cast, strengthening local "base_def" from gimple to gassign *. (gen_conditions_for_pow_int_base): Add a checked cast. --- gcc/ChangeLog.gimple-classes | 7 ++

[gimple-classes, committed 29/44] tree-ssa-dse.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-ssa-dse.c (dse_optimize_stmt): Add checked cast. Replace is_gimple_assign with dyn_cast, introducing local gassign * "assign_stmt", using it in place of "stmt" for typesafety. --- gcc/ChangeLog.gimple-classes | 6 ++ gcc/tree-ssa-d

[gimple-classes, committed 15/44] graphite-sese-to-poly.c: Use gasssign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * graphite-sese-to-poly.c (rewrite_cross_bb_scalar_deps): Add checked cast. (is_reduction_operation_p): Strengthen param from gimple to gassign *. (follow_ssa_with_commutative_ops): Replace is_gimple_assign with a dyn_cas

[gimple-classes, committed 27/44] tree-ssa-dce.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-ssa-dce.c (mark_stmt_if_obviously_necessary): Within case GIMPLE_ASSIGN, introduce local "assign_stmt" and use in place of "stmt" for typesafety. (mark_all_reaching_defs_necessary_1): Add a checked cast. (propagate_necessi

[gimple-classes, committed 23/44] tree-ssa-alias.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * tree-ssa-alias.c (stmt_may_clobber_global_p): Within case GIMPLE_ASSIGN, add a checked cast. (ao_ref_init_from_ptr_and_size): Both the gimple_assign_single_p and is_gimple_assign conditionals require a GIMPLE_ASSIGN, so begin w

[gimple-classes, committed 11/44] gimple-ssa-isolate-paths.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * gimple-ssa-isolate-paths.c (insert_trap_and_remove_trailing_statements): Replace a call to is_gimple_assign with a dyn_cast, introducing a local "assign_stmt", and using it in place of "stmt" for typesafety. --- gcc/ChangeLog.gimple-c

Re: [PATCH][11/n] Merge from match-and-simplify, bit patterns from forwprop

2014-11-06 Thread Marc Glisse
On Thu, 6 Nov 2014, Richard Biener wrote: On Thu, 6 Nov 2014, Marc Glisse wrote: On Thu, 6 Nov 2014, Richard Biener wrote: On Thu, 6 Nov 2014, Marc Glisse wrote: On Thu, 6 Nov 2014, Richard Biener wrote: +/* Try simple folding for X op !X, and X op X with the help + of the truth_valued

[gimple-classes, committed 36/44] cfgexpand.c: Use gassign *

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * cfgexpand.c (add_scope_conflicts_1): Add checked cast. (expand_gimple_cond): Replace check for GIMPLE_ASSIGN with a dyn_cast, introducing local "second_assign", using it in place of "assign" for typesafety. (expand_call_stmt):

[gimple-classes, committed 30/44] gimple-walk.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * gimple-walk.c (walk_gimple_op): Within case GIMPLE_ASSIGN, introduce local gassign * "assign_stmt" and use it in place of "stmt" for typesafety. --- gcc/ChangeLog.gimple-classes | 6 gcc/gimple-walk.c| 74 +++

[gimple-classes, committed 34/44] build_assign and build_type_cast take const gassign *

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * asan.c (asan_expand_check_ifn): Strengthen local "shadow_test" from gimple to gassign *. Introduce gassign * locals "add_7", "cast", "t_ge_shadow", "and_expr", using them in place of gimple_seq_last_stmt (seq) for typesafety.

[gimple-classes, committed 41/44] omp-low.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * omp-low.c (expand_omp_taskreg): Strengthen locals "parcopy_stmt" and "stmt" from gimple to gassign *, replacing a check against GIMPLE_ASSIGN with a dyn_cast. (expand_omp_atomic_fetch_op): Likewise for local "stmt". (expand_omp

[gimple-classes, committed 38/44] ipa-inline-analysis.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * ipa-inline-analysis.c (unmodified_parm): Capture result of gimple_assign_single_p as a new local gassign * "def_assign" and use in place of SSA_NAME_DEF_STMT (op) for typesafety. (unmodified_parm_or_parm_agg_item): Likewise. (e

[gimple-classes, committed 37/44] ipa-split.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * ipa-split.c (find_return_bb): Eliminate check for GIMPLE_ASSIGN since this is covered by gimple_assign_single_p, using the latter to introduce local gassign * "assign_stmt", using it in place of "stmt" for typesafety. (find_ret

[gimple-classes, committed 32/44] Fix issues with gimple_assign_single_p and gimple_assign_load_p

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * asan.c (get_mem_ref_of_assignment): Weaken param "assignment" from const gassign * to gassign *. * gimple.h (gimple_assign_single_p): Eliminate const_gimple variant, since this could call itself, leading to an infinite recursion.

[gimple-classes, committed 42/44] predict.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * predict.c (expr_expected_value_1): Replace is_gimple_assign with a dyn_cast, introducing local "def_assign" and using it in place of "def" for typesafety. --- gcc/ChangeLog.gimple-classes | 6 ++ gcc/predict.c| 13 +++

[gimple-classes, committed 44/44] trans-mem.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * trans-mem.c (thread_private_new_memory): Replace is_gimple_assign with a dyn_cast, introducing local gassign * "assign_stmt", using it in place of "stmt" for typesafety. (examine_assign_tm): Strengthen local "stmt" from gimple to

[gimple-classes, committed 43/44] sese.c: Use gassign

2014-11-06 Thread David Malcolm
gcc/ChangeLog.gimple-classes: * sese.c (rename_uses): Add checked cast to gassign within region guarded by is_gimple_assign. (graphite_copy_stmts_from_block): Likewise. --- gcc/ChangeLog.gimple-classes | 6 ++ gcc/sese.c | 4 ++-- 2 files changed, 8 in

  1   2   >