RE: regs_used estimation in IVOPTS seriously flawed

2014-06-20 Thread Bingfeng Mei
-Original Message- From: Bin.Cheng [mailto:amker.ch...@gmail.com] Sent: 20 June 2014 06:25 To: Bingfeng Mei Cc: gcc@gcc.gnu.org Subject: Re: regs_used estimation in IVOPTS seriously flawed On Tue, Jun 17, 2014 at 10:59 PM, Bingfeng Mei b...@broadcom.com wrote: Hi, I am

Re: regs_used estimation in IVOPTS seriously flawed

2014-06-20 Thread Bin.Cheng
On Fri, Jun 20, 2014 at 5:01 PM, Bingfeng Mei b...@broadcom.com wrote: -Original Message- From: Bin.Cheng [mailto:amker.ch...@gmail.com] Sent: 20 June 2014 06:25 To: Bingfeng Mei Cc: gcc@gcc.gnu.org Subject: Re: regs_used estimation in IVOPTS seriously flawed On Tue, Jun 17, 2014

RE: regs_used estimation in IVOPTS seriously flawed

2014-06-20 Thread Bingfeng Mei
Sorry, typo in previous mail. I also tried counting all SSA names and divide it by a factor. It does NOT seem to work so well -Original Message- From: Bin.Cheng [mailto:amker.ch...@gmail.com] Sent: 20 June 2014 10:19 To: Bingfeng Mei Cc: gcc@gcc.gnu.org Subject: Re: regs_used

Re: regs_used estimation in IVOPTS seriously flawed

2014-06-20 Thread David Edelsohn
On Fri, Jun 20, 2014 at 5:01 AM, Bingfeng Mei b...@broadcom.com wrote: As a short term solution, I tried some simple non-linear functions as Richard suggested to penalize using too many IVs. For example, the following cost in ivopts_global_cost_for_size fixed my regression and actually

Re: Offload Library

2014-06-20 Thread David Edelsohn
On Fri, May 16, 2014 at 7:47 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Dear steering committee, To support the offloading features for Intel's Xeon Phi cards we need to add a foreign library (liboffload) into the gcc repository. README with build instructions is attached. I

Re: Offload Library

2014-06-20 Thread Joel Sherrill
On 6/20/2014 1:46 PM, David Edelsohn wrote: On Fri, May 16, 2014 at 7:47 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Dear steering committee, To support the offloading features for Intel's Xeon Phi cards we need to add a foreign library (liboffload) into the gcc repository.

[Bug tree-optimization/61569] New: faggressive-loop-optimizations overoptimize loop checks with unpredicted result

2014-06-20 Thread n-gcc at nn dot kiev.ua
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61569 Bug ID: 61569 Summary: faggressive-loop-optimizations overoptimize loop checks with unpredicted result Product: gcc Version: 4.8.0 Status: UNCONFIRMED

[Bug lto/61123] With LTO, -fno-short-enums is ignored, resulting in ABI mis-matching in linking.

2014-06-20 Thread Hale.Wang at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61123 --- Comment #6 from Hale Wang Hale.Wang at arm dot com --- Thanks for Zhenqiang Chen's help. This bug is fixed. Refer to the link https://gcc.gnu.org/ml/gcc-patches/2014-06/msg01429.html

[Bug target/61570] New: -march=native CPU you selected does not support x86-64 instruction under QEMU

2014-06-20 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61570 Bug ID: 61570 Summary: -march=native CPU you selected does not support x86-64 instruction under QEMU Product: gcc Version: 4.9.0 Status: UNCONFIRMED Severity:

[Bug middle-end/61571] New: bad aliasing -- wrong FRE

2014-06-20 Thread xinliangli at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61571 Bug ID: 61571 Summary: bad aliasing -- wrong FRE Product: gcc Version: 4.10.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: middle-end

[Bug middle-end/61571] bad aliasing -- wrong FRE

2014-06-20 Thread xinliangli at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61571 --- Comment #1 from davidxl xinliangli at gmail dot com --- Created attachment 32979 -- https://gcc.gnu.org/bugzilla/attachment.cgi?id=32979action=edit source file

[Bug rtl-optimization/61572] New: [4.10 Regression] ICE: in assign_by_spills, at lra-assigns.c:1335

2014-06-20 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61572 Bug ID: 61572 Summary: [4.10 Regression] ICE: in assign_by_spills, at lra-assigns.c:1335 Product: gcc Version: 4.10.0 Status: UNCONFIRMED Severity: normal

[Bug middle-end/61571] bad aliasing -- wrong FRE

2014-06-20 Thread xinliangli at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61571 davidxl xinliangli at gmail dot com changed: What|Removed |Added CC||richard.guenther at

[Bug middle-end/61573] New: [ICE] Segfault while Linux 3.15 build

2014-06-20 Thread kirill.yukhin at intel dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61573 Bug ID: 61573 Summary: [ICE] Segfault while Linux 3.15 build Product: gcc Version: 4.10.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug middle-end/61573] [ICE] Segfault while Linux 3.15 build

2014-06-20 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61573 Markus Trippelsdorf trippels at gcc dot gnu.org changed: What|Removed |Added CC|

[Bug middle-end/61573] [ICE] Segfault while Linux 3.15 build

2014-06-20 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61573 Markus Trippelsdorf trippels at gcc dot gnu.org changed: What|Removed |Added Last reconfirmed|

[Bug middle-end/61573] [4.10 Regression] ICE: Segmentation fault building Linux kernel

2014-06-20 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61573 Markus Trippelsdorf trippels at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW

[Bug middle-end/61573] [4.10 Regression] ICE: Segmentation fault building Linux kernel

2014-06-20 Thread mliska at suse dot cz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61573 Martin Liška mliska at suse dot cz changed: What|Removed |Added CC||mliska at suse dot

[Bug middle-end/61573] [4.10 Regression] ICE: Segmentation fault building Linux kernel

2014-06-20 Thread mliska at suse dot cz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61573 --- Comment #4 from Martin Liška mliska at suse dot cz --- fndecl deletion place: Old value = (tree) 0x76d4a700 New value = (tree) 0xa5a5a5a5a5a5a5a5 memset () at ../sysdeps/x86_64/memset.S:90 90../sysdeps/x86_64/memset.S: No such file

[Bug target/61570] -march=native CPU you selected does not support x86-64 instruction under QEMU

2014-06-20 Thread mikpelinux at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61570 --- Comment #1 from Mikael Pettersson mikpelinux at gmail dot com --- Well, qemu seems confused (as usual). Having lm and lahf_lmin the CPUID feature flags and a 48-bit virtual address space implies being 64-bit, but Intel Family 6 Model 13

[Bug target/61570] -march=native CPU you selected does not support x86-64 instruction under QEMU

2014-06-20 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61570 --- Comment #2 from Jakub Jelinek jakub at gcc dot gnu.org --- Sure, but for this model AFAIK we go through the default: case of PENTIUMPRO anyway, where we only test the flags, so even if it used some completely unknown family number, it would

[Bug ipa/61211] [4.9/4.10 Regression] ICE: verify_cgraph_node failed: edge points to wrong declaration with -O3 -fno-inline

2014-06-20 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61211 --- Comment #3 from Martin Jambor jamborm at gcc dot gnu.org --- Author: jamborm Date: Fri Jun 20 09:54:39 2014 New Revision: 211844 URL: https://gcc.gnu.org/viewcvs?rev=211844root=gccview=rev Log: 2014-06-20 Martin Jambor mjam...@suse.cz

[Bug ipa/61211] [4.9/4.10 Regression] ICE: verify_cgraph_node failed: edge points to wrong declaration with -O3 -fno-inline

2014-06-20 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61211 --- Comment #4 from Martin Jambor jamborm at gcc dot gnu.org --- Author: jamborm Date: Fri Jun 20 09:57:53 2014 New Revision: 211845 URL: https://gcc.gnu.org/viewcvs?rev=211845root=gccview=rev Log: 2014-06-20 Martin Jambor mjam...@suse.cz

[Bug ipa/61211] [4.9/4.10 Regression] ICE: verify_cgraph_node failed: edge points to wrong declaration with -O3 -fno-inline

2014-06-20 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61211 Martin Jambor jamborm at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug target/61570] -march=native CPU you selected does not support x86-64 instruction under QEMU

2014-06-20 Thread mikpelinux at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61570 Mikael Pettersson mikpelinux at gmail dot com changed: What|Removed |Added CC|

[Bug sanitizer/61530] [4.10 Regression] segfault with asan

2014-06-20 Thread chefmax at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61530 --- Comment #7 from Maxim Ostapenko chefmax at gcc dot gnu.org --- Author: chefmax Date: Fri Jun 20 10:26:55 2014 New Revision: 211846 URL: https://gcc.gnu.org/viewcvs?rev=211846root=gccview=rev Log: 2014-06-18 Yury Gribov y.gri...@samsung.com

[Bug ipa/61540] internal compiler error in try_make_edge_direct_virtual_call

2014-06-20 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61540 --- Comment #3 from Martin Jambor jamborm at gcc dot gnu.org --- Author: jamborm Date: Fri Jun 20 11:19:46 2014 New Revision: 211847 URL: https://gcc.gnu.org/viewcvs?rev=211847root=gccview=rev Log: 2014-06-20 Martin Jambor mjam...@suse.cz

[Bug ipa/61540] internal compiler error in try_make_edge_direct_virtual_call

2014-06-20 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61540 --- Comment #4 from Martin Jambor jamborm at gcc dot gnu.org --- Author: jamborm Date: Fri Jun 20 11:26:42 2014 New Revision: 211848 URL: https://gcc.gnu.org/viewcvs?rev=211848root=gccview=rev Log: 2014-06-20 Martin Jambor mjam...@suse.cz

[Bug ipa/61540] internal compiler error in try_make_edge_direct_virtual_call

2014-06-20 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61540 Martin Jambor jamborm at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug libitm/52482] libitm INVALID MNEMONIC in .S (powerpc asm)

2014-06-20 Thread venture37 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52482 venture37 at gmail dot com changed: What|Removed |Added CC||venture37 at gmail dot com

[Bug libitm/52482] libitm INVALID MNEMONIC in .S (powerpc asm)

2014-06-20 Thread venture37 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52482 --- Comment #7 from venture37 at gmail dot com --- This is on 32bit PowerPC 7450 system Darwin 8.11.0 Darwin Kernel Version 8.11.0: Wed Oct 10 18:26:00 PDT 2007; root:xnu-792.24.17~1/RELEASE_PPC Power Macintosh powerpc

[Bug rtl-optimization/61572] [4.10 Regression] ICE: in assign_by_spills, at lra-assigns.c:1335

2014-06-20 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61572 Markus Trippelsdorf trippels at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW

[Bug tree-optimization/61554] [4.10 Regression] ICE: Segmentation fault

2014-06-20 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61554 Markus Trippelsdorf trippels at gcc dot gnu.org changed: What|Removed |Added CC||rguenth

[Bug target/61407] Build errors on latest OS X 10.10 Yosemite with Xcode 6 on GCC 4.8.3

2014-06-20 Thread egall at gwmail dot gwu.edu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61407 Eric Gallager egall at gwmail dot gwu.edu changed: What|Removed |Added CC||egall at

[Bug regression/61510] [4.10 Regression]: 20_util/scoped_allocator/requirements/explicit_instantiation.cc and tr1/6_containers/tuple/requirements/explicit_instantiation.cc

2014-06-20 Thread jgreenhalgh at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61510 --- Comment #4 from jgreenhalgh at gcc dot gnu.org --- Obviously something simple like this will fix the problem, but I don't know the area to say that it doesn't just mask a deeper bug. Honza? --- 2014-06-19 James Greenhalgh

[Bug c++/61121] -ftree-parallelize-loops=n (n as value) not accepted in 4.9.0

2014-06-20 Thread jmichae3 at yahoo dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61121 Jim Michaels jmichae3 at yahoo dot com changed: What|Removed |Added Status|RESOLVED|UNCONFIRMED

[Bug c++/61121] -O2 -ftree-parallelize-loops=0 for maximum not accepted in 4.9.0

2014-06-20 Thread jmichae3 at yahoo dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61121 Jim Michaels jmichae3 at yahoo dot com changed: What|Removed |Added Summary|-ftree-parallelize-loops=n |-O2

[Bug c++/61121] -O2 -ftree-parallelize-loops=0 for maximum not accepted in 4.9.0

2014-06-20 Thread jmichae3 at yahoo dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61121 --- Comment #7 from Jim Michaels jmichae3 at yahoo dot com --- http://msdn.microsoft.com/en-us/library/windows/desktop/dd405485%28v=vs.85%29.aspx http://msdn.microsoft.com/en-us/library/windows/desktop/ms679351%28v=vs.85%29.aspx

[Bug c++/61568] unscoped enums different types differ from __underlying_type

2014-06-20 Thread harald at gigawatt dot nl
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61568 Harald van Dijk harald at gigawatt dot nl changed: What|Removed |Added CC||harald at

[Bug c++/61574] New: Confusing .debug_line generation by -g option

2014-06-20 Thread jamesgua at ca dot ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61574 Bug ID: 61574 Summary: Confusing .debug_line generation by -g option Product: gcc Version: 4.8.2 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug c++/61556] [c++11][4.9/4.10 Regression] ‘*(const ValueType*)this’ is not a constant expression with valid code

2014-06-20 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61556 --- Comment #1 from Jason Merrill jason at gcc dot gnu.org --- Author: jason Date: Fri Jun 20 18:31:53 2014 New Revision: 211853 URL: https://gcc.gnu.org/viewcvs?rev=211853root=gccview=rev Log: PR c++/61556 * call.c (build_over_call):

[Bug c++/59296] [c++11] ref-qualified member function is ambiguous

2014-06-20 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59296 --- Comment #6 from Jason Merrill jason at gcc dot gnu.org --- Author: jason Date: Fri Jun 20 18:32:35 2014 New Revision: 211854 URL: https://gcc.gnu.org/viewcvs?rev=211854root=gccview=rev Log: PR c++/59296 * call.c

[Bug c++/60265] [C++11] using-declaration of enumerator fails if fully qualified

2014-06-20 Thread ppluzhnikov at google dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60265 Paul Pluzhnikov ppluzhnikov at google dot com changed: What|Removed |Added CC|

[Bug c++/59296] [c++11] ref-qualified member function is ambiguous

2014-06-20 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59296 --- Comment #7 from Jason Merrill jason at gcc dot gnu.org --- Author: jason Date: Fri Jun 20 18:35:19 2014 New Revision: 211856 URL: https://gcc.gnu.org/viewcvs?rev=211856root=gccview=rev Log: PR c++/59296 * call.c

[Bug c++/61556] [c++11][4.9/4.10 Regression] ‘*(const ValueType*)this’ is not a constant expression with valid code

2014-06-20 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61556 --- Comment #2 from Jason Merrill jason at gcc dot gnu.org --- Author: jason Date: Fri Jun 20 18:34:37 2014 New Revision: 211855 URL: https://gcc.gnu.org/viewcvs?rev=211855root=gccview=rev Log: PR c++/61556 * call.c (build_over_call):

[Bug c++/61568] unscoped enums different types differ from __underlying_type

2014-06-20 Thread myriachan at cox dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61568 --- Comment #2 from Melissa myriachan at cox dot net --- (In reply to Harald van Dijk from comment #1) The C++ standard requires an unscoped enumeration type with an underlying type that is not fixed to promote to signed int, if signed int is

[Bug c++/61574] Confusing .debug_line generation by -g option

2014-06-20 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61574 --- Comment #1 from Andrew Pinski pinskia at gcc dot gnu.org --- This is deconstructors.

[Bug fortran/33363] gfortran is missing -fcase-preserve option

2014-06-20 Thread fxcoudert at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=33363 --- Comment #1 from Francois-Xavier Coudert fxcoudert at gcc dot gnu.org --- Author: fxcoudert Date: Fri Jun 20 20:11:30 2014 New Revision: 211858 URL: https://gcc.gnu.org/viewcvs?rev=211858root=gccview=rev Log: PR fortran/33363 *

[Bug fortran/33363] gfortran is missing -fcase-preserve option

2014-06-20 Thread fxcoudert at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=33363 Francois-Xavier Coudert fxcoudert at gcc dot gnu.org changed: What|Removed |Added Status|NEW

[Bug bootstrap/61565] [4.10 Regression] ICE building libjava/interpret.cc

2014-06-20 Thread rth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61565 --- Comment #3 from Richard Henderson rth at gcc dot gnu.org --- Patch: https://gcc.gnu.org/ml/gcc-patches/2014-06/msg01637.html

[Bug libfortran/37634] Fix libgfortran compiling to support GCC_NO_EXECUTABLES

2014-06-20 Thread fxcoudert at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=37634 Francois-Xavier Coudert fxcoudert at gcc dot gnu.org changed: What|Removed |Added Status|NEW

[Bug c++/61575] New: [4.9 Regression] Bogus invalid initialization of reference of type 'const D' from expression of type 'brace-enclosed initializer list'

2014-06-20 Thread ppluzhnikov at google dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61575 Bug ID: 61575 Summary: [4.9 Regression] Bogus invalid initialization of reference of type 'const D' from expression of type 'brace-enclosed initializer list' Product: gcc

[Bug c++/61575] [4.9 Regression] Bogus invalid initialization of reference of type 'const D' from expression of type 'brace-enclosed initializer list'

2014-06-20 Thread ppluzhnikov at google dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61575 --- Comment #1 from Paul Pluzhnikov ppluzhnikov at google dot com --- Sorry, cut/paste error. Both gcc-svn-4_9-r211828/bin/gcc gcc-svn-4_9-r211175/bin/gcc fails the same way.

[Bug target/52268] tls support should be added for darwin11

2014-06-20 Thread egall at gwmail dot gwu.edu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52268 Eric Gallager egall at gwmail dot gwu.edu changed: What|Removed |Added CC||egall at

[Bug debug/59051] DW_tag_restrict_type not used

2014-06-20 Thread mark at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59051 Mark Wielaard mark at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW Last

[Bug tree-optimization/61576] New: wrong code at -O3 on x86_64-linux-gnu

2014-06-20 Thread su at cs dot ucdavis.edu
version 4.10.0 20140620 (experimental) [trunk revision 211848] (GCC) $ $ gcc-trunk -O2 small.c; a.out 0 $ gcc-4.9.0 -O3 small.c; a.out 0 $ $ gcc-trunk -O3 small.c; a.out 1 $ int printf (const char *, ...); volatile int a, b; int c, d, e, f; static int fn1

[Bug c++/61538] g++ compiled binary, linked to glibc libpthread, hangs on SGI MIPS R1x000 systems on Linux

2014-06-20 Thread kumba at gentoo dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61538 --- Comment #10 from Joshua Kinard kumba at gentoo dot org --- I rebuilt both glibc-2.19 and gcc-4.8.3 w/ debugging, though gcc's build system managed to strip out or optimize away some of the debugging code. That said, it's enough to see that

[Bug c++/61538] g++ compiled binary, linked to glibc libpthread, hangs on SGI MIPS R1x000 systems on Linux

2014-06-20 Thread kumba at gentoo dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61538 --- Comment #11 from Joshua Kinard kumba at gentoo dot org --- I also have another test case from glibc itself, where when compiling glibc-2.19 w/ gcc-4.8.x or greater, at the end, it creates a statically-linked version of 'ln' as 'sln', and

Re: [PATCH, ARM] MI-thunk fix for TARGET_THUMB1_ONLY

2014-06-20 Thread Chung-Lin Tang
On 2014/6/18 上午 06:26, Ramana Radhakrishnan wrote: On Sun, Jun 8, 2014 at 12:27 PM, Chung-Lin Tang clt...@codesourcery.com wrote: Hi Richard, Ramana, Attached is a small fix for resolving a g++.old-deja/g++.jason/thunk2.C regression we found under a TARGET_THUMB1_ONLY multilib (-mthumb

Re: [PATCH, ARM] Cortex-A9 MPCore volatile load workaround

2014-06-20 Thread Chung-Lin Tang
Ping. On 2014/6/9 10:03 PM, Chung-Lin Tang wrote: Hi Richard, As we talked about earlier, here's a patch to add a compiler option to work around Cortex-A9 MPCore errata 761319: http://infocenter.arm.com/help/topic/com.arm.doc.uan0004a/UAN0004A_a9_read_read.pdf What the option does

Move DECL_INIT_PRIORITY/FINI_PRIORITY to symbol table

2014-06-20 Thread Jan Hubicka
Hi, this patch moves init and fini priorities to symbol table instead of trees. They are already in on-side hashtables, but the hashtables are now maintaned by symbol table. This is needed for correctness with LTO. Currently tree merging may load declaration with priority and then ggc_free it

Delay RTL initialization until it is really needed

2014-06-20 Thread Jan Hubicka
Hi, IRA initialization shows high in profiles even when building lto objects. This patch simply delays RTL backend initialization until we really decide to output a function. In some cases this avoids the initialization completely (like in the case of LTO but also user target attributes) and

Re: [PATCH][AArch64] Fix some saturating math NEON intrinsics types

2014-06-20 Thread Marcus Shawcroft
On 16 June 2014 15:26, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, I noticed that a few saturating math intrinsics in arm_neon.h for aarch64 have the wrong types, i.e. not what's mandated by the ACLE spec. This patch fixes that by adjusting the types of the builtin functions that

Re: [PATCH] Implement -fsanitize=bounds and internal calls in FEs

2014-06-20 Thread Marek Polacek
On Thu, Jun 19, 2014 at 07:19:31PM +0200, Jakub Jelinek wrote: + case IFN_UBSAN_BOUNDS: + ubsan_expand_bounds_btn (gsi); + break; default: Why *_btn instead of *_ifn ? Remnant from when I was using __builtin.ubsan instead of the internal call.

Re: [PATCH] Implement -fsanitize=bounds and internal calls in FEs

2014-06-20 Thread Jakub Jelinek
On Fri, Jun 20, 2014 at 10:43:04AM +0200, Marek Polacek wrote: + +/* Internal function code. */ +ENUM_BITFIELD(internal_fn) ifn : 5; Any reason for the : 5 here? I mean, the union also contains unsigned int, so it doesn't hurt if you use full 32 bits for it there, and it should be

[linaro/gcc-4_9-branch] AArch64 costs model backports

2014-06-20 Thread Yvan Roux
Hi all, we have backported a set of AArch64 costs model related revisions in the linaro/gcc-4_9-branch at r211843. The backported revisions are: 210493 : [AArch64 costs 1/18] Refactor aarch64_address_costs. 210494 : [AArch64 costs 2/18] Add cost tables for Cortex-A57 210495 : [AArch64 costs

Re: [PATCH] Implement -fsanitize=bounds and internal calls in FEs

2014-06-20 Thread Marek Polacek
On Fri, Jun 20, 2014 at 10:57:47AM +0200, Jakub Jelinek wrote: On Fri, Jun 20, 2014 at 10:43:04AM +0200, Marek Polacek wrote: + +/* Internal function code. */ +ENUM_BITFIELD(internal_fn) ifn : 5; Any reason for the : 5 here? I mean, the union also contains unsigned int, so it

Re: [PATCH] Implement -fsanitize=bounds and internal calls in FEs

2014-06-20 Thread Jakub Jelinek
On Fri, Jun 20, 2014 at 11:34:26AM +0200, Marek Polacek wrote: On Fri, Jun 20, 2014 at 10:57:47AM +0200, Jakub Jelinek wrote: On Fri, Jun 20, 2014 at 10:43:04AM +0200, Marek Polacek wrote: + +/* Internal function code. */ +ENUM_BITFIELD(internal_fn) ifn : 5; Any reason

Re: [RFC] optimize x - y cmp 0 with undefined overflow

2014-06-20 Thread Eric Botcazou
[I'm at last back to this...] With [1, -x + INF] as the resulting range? But it can be bogus if x is itself equal to +INF (unlike the input range [x + 1, +INF] which is always correct) Hmm, indeed. so this doesn't look valid to me. I don't see how we can get away without a

Re: [PATCH] Implement -fsanitize=bounds and internal calls in FEs

2014-06-20 Thread Marek Polacek
On Fri, Jun 20, 2014 at 11:39:23AM +0200, Jakub Jelinek wrote: On Fri, Jun 20, 2014 at 11:34:26AM +0200, Marek Polacek wrote: On Fri, Jun 20, 2014 at 10:57:47AM +0200, Jakub Jelinek wrote: On Fri, Jun 20, 2014 at 10:43:04AM +0200, Marek Polacek wrote: + +/* Internal function

[PATCH] Fix ICE in Asan

2014-06-20 Thread Maxim Ostapenko
Hi, This patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61530. Tested on x86_64-unknown-linux-gnu, no regressions. Ok to commit? -Maxim gcc/ChangeLog: 2014-06-20 Yury Gribov y.gri...@samsung.com Max Ostapenko m.ostape...@partner.samsung.com * asan.c (build_check_stmt):

Re: [PATCH] Fix ICE in Asan

2014-06-20 Thread Jakub Jelinek
On Fri, Jun 20, 2014 at 02:04:25PM +0400, Maxim Ostapenko wrote: This patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61530. Tested on x86_64-unknown-linux-gnu, no regressions. Ok to commit? -Maxim gcc/ChangeLog: 2014-06-20 Yury Gribov y.gri...@samsung.com

Re: [PATCH] Fix ICE in Asan

2014-06-20 Thread Maxim Ostapenko
On 06/20/2014 02:07 PM, Jakub Jelinek wrote: On Fri, Jun 20, 2014 at 02:04:25PM +0400, Maxim Ostapenko wrote: This patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61530. Tested on x86_64-unknown-linux-gnu, no regressions. Ok to commit? -Maxim gcc/ChangeLog: 2014-06-20 Yury Gribov

RE: [PATCH] Fix PR61375: cancel bswap optimization when value doesn't fit in a HOST_WIDE_INT

2014-06-20 Thread Thomas Preud'homme
From: Richard Biener [mailto:richard.guent...@gmail.com] Sent: Tuesday, June 10, 2014 5:05 PM Backports are welcome - please post a patch. Sorry for the delay. Here you are: diff --git a/gcc/testsuite/gcc.c-torture/execute/pr61375.c b/gcc/testsuite/gcc.c-torture/execute/pr61375.c new

[PATCH] Fix for invalid sanitization of trailing byte in __builtin_strlen

2014-06-20 Thread Maxim Ostapenko
This patch fixes invalid sanitization of trailing byte in __builtin_strlen ( https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61547). Tested on x86_64-unknown-linux-gnu, no regressions. Ok to commit? -Maxim gcc/ChangeLog: 2014-06-20 Yury Gribov y.gri...@samsung.com Max Ostapenko

[PATCH 2/N] allow storing values directly in hash tables

2014-06-20 Thread tsaunders
From: Trevor Saunders tsaund...@mozilla.com Hi, this patch allows you to define the type the hash table stores as elements instead of the type elements point at by having your hash descriptor define the type store_values_directly. It turns out trying to implement both cases with the same code

[PATCH 3/3] add hash_map class

2014-06-20 Thread tsaunders
From: Trevor Saunders tsaund...@mozilla.com Hi, This patch adds a hash_map class so we can consolidate the boiler plate around using hash_table as a map, it also allows us to get rid of pointer_map which I do in this patch by converting its users to hash_map. bootstrapped + regtested without

Re: Move DECL_INIT_PRIORITY/FINI_PRIORITY to symbol table

2014-06-20 Thread Trevor Saunders
On Fri, Jun 20, 2014 at 08:41:22AM +0200, Jan Hubicka wrote: Hi, this patch moves init and fini priorities to symbol table instead of trees. They are already in on-side hashtables, but the hashtables are now maintaned by symbol table. This is needed for correctness with LTO. Currently tree

Re: [PATCH] Fix for invalid sanitization of trailing byte in __builtin_strlen

2014-06-20 Thread Jakub Jelinek
On Fri, Jun 20, 2014 at 02:49:12PM +0400, Maxim Ostapenko wrote: This patch fixes invalid sanitization of trailing byte in __builtin_strlen ( https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61547). Tested on x86_64-unknown-linux-gnu, no regressions. + +#include sanitizer/asan_interface.h +

Re: [PATCH] Fix for invalid sanitization of trailing byte in __builtin_strlen

2014-06-20 Thread Yuri Gribov
On Fri, Jun 20, 2014 at 2:57 PM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Jun 20, 2014 at 02:49:12PM +0400, Maxim Ostapenko wrote: This patch fixes invalid sanitization of trailing byte in __builtin_strlen ( https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61547). Tested on

Re: [PATCH] Change default for --param allow-...-data-races to off

2014-06-20 Thread Martin Jambor
Hi, On Thu, Jun 19, 2014 at 06:18:47PM +0200, Bernd Edlinger wrote: Hi, from a recent discussion on g...@gcc.gnu.org I have learned that the default of --param allow-store-data-races is still 1, and it is causing problems. Therefore I would like to suggest to change the default of this

Re: [PATCH] dwarf2out.c: Pass DWARF type modifiers around as flags argument.

2014-06-20 Thread Mark Wielaard
While adding some new type modifiers I did find a typo in my original patch, so I decided to add some testcases to make sure no regressions were introduced. This is the same patch as the original, but with the typo in modified_type_die fixed that could accidentally add an extra layer of type

Re: [PATCH AArch64 1/2] PR/60825 Make float64x1_t in arm_neon.h a proper vector type

2014-06-20 Thread Marcus Shawcroft
On 19 June 2014 13:27, Alan Lawrence alan.lawre...@arm.com wrote: This updates the .md files to generate V1DFmode patterns instead of DFmode for create and reinterpret, and the corresponding __builtins. The various other float64x1_t intrinsics can then be rewritten, generally I've tried to

Re: [PATCH AArch64 2/2] PR/60825 Make {int,uint}64x1_t in arm_neon.h a proper vector type

2014-06-20 Thread Marcus Shawcroft
On 19 June 2014 13:30, Alan Lawrence alan.lawre...@arm.com wrote: Similarly, this makes int64x1_t a proper vector type, updating arm_neon.h with many explicit vector construction/destruction operations (also including some range checking using __builtin_aarch64_im_lane_boundsi). Change the

Re: [PATCH] Fix for invalid sanitization of trailing byte in __builtin_strlen

2014-06-20 Thread Maxim Ostapenko
What about bootstrap though? Bootstrap in progress. +__SIZE_TYPE__ strlen (const char *p) { + /* Simulate error */ + if (p == a) +return 1; Why this? Can't you instead just use __attribute__((no_sanitize_address, noinline)) on it instead? Done. Ok to commit if bootstrap will

Re: [PATCH] Fix for invalid sanitization of trailing byte in __builtin_strlen

2014-06-20 Thread Jakub Jelinek
On Fri, Jun 20, 2014 at 04:55:43PM +0400, Maxim Ostapenko wrote: What about bootstrap though? Bootstrap in progress. +__SIZE_TYPE__ strlen (const char *p) { + /* Simulate error */ + if (p == a) +return 1; Why this? Can't you instead just use

Re: C++ PATCH for c++/59296 (rvalue object and lvalue ref-qualifier)

2014-06-20 Thread Jason Merrill
On 06/19/2014 01:29 PM, Marc Glisse wrote: That looks weird to me. The const version is a better match than the const, so we should pick that one in overload resolution, but if we remove the const version, the other one seems valid to me Hmm, you're right, I was confused. Jason

Re: [PATCH] Fix for invalid sanitization of trailing byte in __builtin_strlen

2014-06-20 Thread Maxim Ostapenko
On 06/20/2014 04:59 PM, Jakub Jelinek wrote: On Fri, Jun 20, 2014 at 04:55:43PM +0400, Maxim Ostapenko wrote: What about bootstrap though? Bootstrap in progress. +__SIZE_TYPE__ strlen (const char *p) { + /* Simulate error */ + if (p == a) +return 1; Why this? Can't you instead just

Re: [PATCH, Testsuite, AArch64] Make aapcs64.exp Tests Big-Endian Friendly

2014-06-20 Thread Marcus Shawcroft
On 19 June 2014 14:32, Yufeng Zhang yufeng.zh...@arm.com wrote: Hi, This patch updates a number of aapcs64 tests to make them big-endian friendly. Changes are mainly: * checking the W regs instead of X regs for integral arguments less than 8 bytes * correcting the corresponding stack

Re: [RFC] Add a .gitattributes file for use with git-merge-changelog

2014-06-20 Thread Tom Tromey
Samuel == Samuel Bronson naes...@gmail.com writes: Samuel [Am I really supposed to CC this to gcc@ like binutils/MAINTAINERS Samuel says I should?] I think just for files that are intended to be put in both trees and shared. Samuel Individual users will still have to: Samuel 1. Install

Re: [PATCH][AArch64] Fix some saturating math NEON intrinsics types

2014-06-20 Thread Kyrill Tkachov
On 20/06/14 09:41, Marcus Shawcroft wrote: On 16 June 2014 15:26, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, I noticed that a few saturating math intrinsics in arm_neon.h for aarch64 have the wrong types, i.e. not what's mandated by the ACLE spec. This patch fixes that by adjusting

[PATCH][AArch64] Implement vfma_f64, vmla_f64, vfms_f64, vmls_f64 intrinsics

2014-06-20 Thread Kyrill Tkachov
Hi all, Now that Alan fixed the float64x1_t machinery, this patch implements some low-hanging intrinsics in arm_neon.h. Tested aarch64-none-elf and bootstrapped on aarch64-linux. Ok for trunk? Thanks, Kyrill 2014-06-20 Kyrylo Tkachov kyrylo.tkac...@arm.com *

[linaro/gcc-4_8-branch] Merge from gcc-4_8-branch

2014-06-20 Thread Yvan Roux
Hi, we have merged the gcc-4_8-branch into linaro/gcc-4_8-branch up to revision 210799 as r211850. Thanks Yvan

Re: [PATCH] Implement -fsanitize=bounds and internal calls in FEs

2014-06-20 Thread Joseph S. Myers
I don't have any comments on this patch. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH 5/5] add libcc1

2014-06-20 Thread Tom Tromey
Trevor == Trevor Saunders tsaund...@mozilla.com writes: Trevor I'm curious, what is the reason you choose not to write this in C++11 or Trevor later? Its distributed with gcc, so the only case where you aren't Trevor building with the in tree compiler and libraries is when your cross Trevor

Re: [PATCH] Implement -fsanitize=bounds and internal calls in FEs

2014-06-20 Thread Marek Polacek
On Thu, Jun 19, 2014 at 07:47:54PM +0200, Jakub Jelinek wrote: On Thu, Jun 19, 2014 at 04:56:53PM +0200, Marek Polacek wrote: + /* Don't instrument this FMA-like array in non-strict Also, please don't use FMA to mean flexible member array, it is flexible array member, but more

Re: [patch i386]: Combine memory and indirect jump

2014-06-20 Thread Kai Tietz
I tested variant to use additional the split pass for memory combining for indirect jumps. By this we don't need to add a second peephole2 pass. Other advantage of this version is that even for -O1 we do combining. ChangeLog 2014-06-20 Kai Tietz kti...@redhat.com PR target/39284 *

[PATCH] Remove bogus include path with in-tree cloog

2014-06-20 Thread Bernd Edlinger
Hi, I have noticed there is a minor flaw with the include path when cloog is installed in-tree. That is, the cloog-include directory is added twice, first with absolute path, and then again with relative path, but with one ../ to little, so this is useless when compiling sources in the gcc

Re: [patch i386]: Combine memory and indirect jump

2014-06-20 Thread Richard Henderson
On 06/20/2014 08:56 AM, Kai Tietz wrote: +(define_split + [(set (match_operand:W 0 register_operand) +(match_operand:W 1 memory_operand)) + (set (pc) (match_dup 0))] + !TARGET_X32 peep2_reg_dead_p (2, operands[0]) + [(set (pc) (match_dup 1))]) + Huh? You can't use peep2 data

[PATCH] Fix arrays in rtx.u + add minor rtx verification

2014-06-20 Thread Marek Polacek
When implementing -fsanitize=bounds I noticed a whole slew of errors about accessing u.fld[] field in rtx_def. Turned out this is indeed a bug, the array should have a size of 8; u.hwint[] array had similar issue. Thus fixed, plus I added some verification code to genpreds.c (can't do it in

  1   2   >