Re: [PATCH 007/236] New function: for_each_rtx_in_insn

2014-08-14 Thread Richard Sandiford
David Malcolm writes: > On Tue, 2014-08-12 at 15:08 -0600, Jeff Law wrote: >> On 08/06/14 11:19, David Malcolm wrote: >> > gcc/ >> >* rtl.h (for_each_rtx_in_insn): New function. >> >* rtlanal.c (for_each_rtx_in_insn): Likewise. >> OK. Note that we're moving away from for_each_rtx... I h

Re: [C++ PATCH] PR c++/62101

2014-08-14 Thread Jason Merrill
On 08/13/2014 08:48 PM, Ville Voutilainen wrote: Ok, modified patch attached. The changelog entry was attached to the previous mail as an evil binary attachment (gmail...) Ah, so it was. here, for convenience: Thanks; having it in the body of the mail is more convenient for me as 'git am'

[PATCH] Avoid redundant indirect_info computation during inderct edge cloning

2014-08-14 Thread Ilya Enkovich
Hi, I get a segafult in decl_maybe_in_construction_p during function versioning. We have following steps in clone creation (e.g. as in create_version_clone_with_body): 1. Create function decl 2. Create clone of cgraph node 3. Copy function body After the first step there is no body attached

PR62091 (ipa-devirt disagreeing with ipa-prop)

2014-08-14 Thread Jan Hubicka
Hi, testcase in PR 62092 triggers assert I added to temporarily check that new ipa-devirt code handles all cases as ipa-prop devirtualization. In this case ipa-prop gives wrong answer because of bug I introduced into walk_aliased_vdefs. The function_entry_reached is supposed to be cleared at begg

Re: [testsuite patch] add __ARM_ARCH check for arm_v8_neon_ok

2014-08-14 Thread Janis Johnson
On 08/14/2014 01:06 AM, Ramana Radhakrishnan wrote: > On Mon, Aug 11, 2014 at 11:01 PM, Janis Johnson > wrote: >> The check for effective target arm_v8_neon_ok passes even if __ARM_ARCH >> is not 8 or greater, but then some tests fail because intrinsic functions >> used in the test have not been d

[rl78] allow const->far moves

2014-08-14 Thread DJ Delorie
Minor tweak, committed. * config/rl78/rl78-virt.md (movhi_virt): Allow const->far moves. Index: config/rl78/rl78-virt.md === --- config/rl78/rl78-virt.md(revision 213994) +++ config/rl78/rl78-virt.md(working copy) @@

RE: [PATCH] Fix confusion between target, host and symbolic number byte sizes

2014-08-14 Thread Thomas Preud'homme
Ping? Best regards, Thomas > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme > Sent: Thursday, August 07, 2014 1:57 PM > To: gcc-patches@gcc.gnu.org > Subject: RE: [PATCH] Fix confusion between target, hos

Re: [PATCH testcase]fix failure of g++.dg/ext/arm-fp16/fp16-mangle-1.C

2014-08-14 Thread Bin.Cheng
On Thu, Aug 14, 2014 at 11:18 PM, Jason Merrill wrote: > On 08/14/2014 04:31 AM, Bin Cheng wrote: >> >> g++.dg/ext/arm-fp16/fp16-mangle-1.C is failed because GCC now sets >> DECL_COMDAT on template instantiations if flag_implicit_templates is in >> effect. Then DECL_WEAK will be set accordingly.

[rl78] disable umul for G10

2014-08-14 Thread DJ Delorie
The G10 family doesn't support this opcode. Committed. * config/rl78/rl78-expand.md (umulqihi3): Disable for G10. * config/rl78/rl78-virt.md (umulhi3_shift_virt): Likewise. (umulqihi3_virt): Likewise. * config/rl78/rl78-real.md (umulhi3_shift_real): Likewise.

[rl78] split mem-mem moves

2014-08-14 Thread DJ Delorie
Letting GCC think that any mem-mem alternative is OK leads to trouble with far mem to far mem moves, so split out the moves we can make. Committed. * config/rl78/predicates.md (rl78_near_mem_operand): New. * config/rl78/rl78-virt.md (movqi_virt_mm, movqi_virt) (movhi_virt_

[rl78] allow global+acc

2014-08-14 Thread DJ Delorie
Minor optimization. Committed. * config/rl78/rl78-real.md (addqi3_real): Allow adding global variables to the accumulator. Index: config/rl78/rl78-real.md === --- config/rl78/rl78-real.md(revision 213996) +++ co

[PING] [PATCH 1/2] Add -B support to gcc-ar/ranlib/nm

2014-08-14 Thread Andi Kleen
Andi Kleen writes: Ping! > From: Andi Kleen > > To use gcc-{ar,ranlib} for boot strap we need to add a -B option > to the tool. Since ar has weird and unusual argument conventions > implement the code by hand instead of using any libraries. > > v2: Fix typo > > gcc/: > > 2014-08-04 Andi Kleen

[PATCH] cygwin: accept -pthread

2014-08-14 Thread Yaakov Selkowitz
The attached patch enables the -pthread flag for cygwin targets. While not strictly necessary for compiling or linking with pthread or _r functions on Cygwin, accepting it allows for better compatibility with other platforms. -- Yaakov Selkowitz Associate Software Engineer, ARM Red Hat, Inc.

Re: [PATCH 007/236] New function: for_each_rtx_in_insn

2014-08-14 Thread Jeff Law
On 08/14/14 15:36, Richard Sandiford wrote: Right. I think they're held up on patch 40 (ironically the one that conflicts with yours). I think we could declare side effects in notes as invalid and add some ENABLE_CHECKING bits to enforce that. With those in place, my concerns around #40 from

Re: [PATCH 156/236] PHASE 4: Removal of scaffolding

2014-08-14 Thread Jeff Law
On 08/06/14 11:22, David Malcolm wrote: / * rtx-classes-status.txt: Update. --- rtx-classes-status.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/rtx-classes-status.txt b/rtx-classes-status.txt index b22cb1e..90d6efd 100644 --- a/rtx-classes-status.txt +++

Re: [PATCH lto]Fix the mis-matched arguments of lto_define_builtins

2014-08-14 Thread Bin.Cheng
On Fri, Aug 8, 2014 at 4:54 PM, Richard Biener wrote: > On Fri, Aug 8, 2014 at 10:05 AM, Bin.Cheng wrote: >> On Thu, Aug 7, 2014 at 8:06 PM, Richard Biener >> wrote: >>> On Thu, Aug 7, 2014 at 11:46 AM, Bin Cheng wrote: Hi, As analyzed in PR62032, this patch fixes the latent lto bug b

Re: [PATCH 2/3]Improve induction variable elimination

2014-08-14 Thread Bin.Cheng
On Thu, Aug 14, 2014 at 11:29 PM, Sebastian Pop wrote: > Bin.Cheng wrote: >> >> The overflow check can be improved by using deeper inspection to prove the >> >> equality. This patch deals with that by making below two improvements: >> >> a) Handles constant cases. >> >> b) Uses affine expansi

Re: [PATCH] Asan static optimization (draft)

2014-08-14 Thread Yuri Gribov
On Thu, Aug 14, 2014 at 8:53 PM, Konstantin Serebryany wrote: > In order for your work to be generally useful, I'd ask several things: > - Update > https://code.google.com/p/address-sanitizer/wiki/CompileTimeOptimizations > with examples that will be handled Done (to be honest I only plan to do

<    1   2