[Bug c/66249] New: -Wformat-signedness should not warn on enums

2015-05-21 Thread eblake at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66249 Bug ID: 66249 Summary: -Wformat-signedness should not warn on enums Product: gcc Version: unknown Status: UNCONFIRMED Severity: normal Priority: P3 Component: c

Re: [PATCH][DRIVER] Wrong C++ include paths when configuring with --with-sysroot=/

2015-05-21 Thread Joseph Myers
On Thu, 21 May 2015, Yvan Roux wrote: There is this old patch submitted by Matthias on that same issue, if its logic is the right one for you Joseph I can rebase/validate it Joseph. https://gcc.gnu.org/ml/gcc-patches/2012-02/msg00320.html Yes, that seems better. -- Joseph S. Myers

[Bug c/65446] Improve -Wformat-signedness

2015-05-21 Thread eblake at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65446 --- Comment #3 from Eric Blake eblake at redhat dot com --- see also bug 66249 where the implementation-defined signedness of enums comes into play, and where I argue that neither %d nor %u should warn when an enum type is passed through varargs

Re: [PATCH][ARM] Handle UNSPEC_VOLATILE in rtx costs and don't recurse inside the unspec

2015-05-21 Thread Kyrill Tkachov
Ping^3. Thanks, Kyrill On 12/05/15 10:08, Kyrill Tkachov wrote: Ping^2. Thanks, Kyrill On 30/04/15 13:01, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01047.html Thanks, Kyrill On 20/04/15 17:28, Kyrill Tkachov wrote: Hi all, A pet project of mine is to get to

Re: [PATCH][ARM/AArch64] Properly cost rev16 operand

2015-05-21 Thread Kyrill Tkachov
Ping. Thanks, Kyrill On 12/05/15 10:07, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2015-05/msg00013.html Thanks, Kyrill On 01/05/15 09:24, Kyrill Tkachov wrote: Hi all, It occurs to me that in the IOR-of-shifts form of the rev16 operation we should be costing the operand

[Bug c/66249] -Wformat-signedness should not warn on enums

2015-05-21 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66249 Marek Polacek mpolacek at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW Last

Re: [PATCH][ARM][stage-1] Initialise cost to COSTS_N_INSNS (1) and increment in arm rtx costs

2015-05-21 Thread Kyrill Tkachov
Ping^3. Thanks, Kyrill On 12/05/15 10:09, Kyrill Tkachov wrote: Ping^2. Thanks, Kyrill On 30/04/15 13:00, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01130.html Thanks, Kyrill On 21/04/15 10:11, Kyrill Tkachov wrote: Hi all, This is the first of a series to

Re: [PATCH] PR target/66232: -fPIC -fno-plt -mx32 fails to generate indirect branch via GOT

2015-05-21 Thread H.J. Lu
On Thu, May 21, 2015 at 6:33 AM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, May 21, 2015 at 6:11 AM, Uros Bizjak ubiz...@gmail.com wrote: On Thu, May 21, 2015 at 2:59 PM, H.J. Lu hjl.to...@gmail.com wrote: X32 doesn't support indirect branch via 32-bit memory slot since indirect branch will

Re: Check canonical types in verify_type

2015-05-21 Thread Jan Hubicka
Hmm, I see, interesting hack. For the first part of comment, I see that qualifiers needs to be ignored, but I do not see why we put short * and int * pointers to same class. For the reason that people are very lazy. For example GCC has code punning void ** and void * and void * to Type

[Bug c/66247] make check-gmp fails for gcc-5.1.0

2015-05-21 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66247 Jakub Jelinek jakub at gcc dot gnu.org changed: What|Removed |Added CC||jakub at gcc dot

Re: [c++std-parallel-1632] Re: Compilers and RCU readers: Once more unto the breach!

2015-05-21 Thread Michael Matz
Hi, On Thu, 21 May 2015, Paul E. McKenney wrote: The point is -exactly- to codify the current state of affairs. Ah, I see, so it's not yet about creating a more useful (for compilers, that is) model. char * fancy_assign (char *in) { return in; } ... char *x, *y; x =

[Bug c/60194] -Wformat should also warn when using %d (instead of %u) for unsigned arguments

2015-05-21 Thread eblake at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60194 Eric Blake eblake at redhat dot com changed: What|Removed |Added CC||eblake at redhat

[Bug c/66247] make check-gmp fails for gcc-5.1.0

2015-05-21 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66247 Marek Polacek mpolacek at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

[Bug rtl-optimization/66248] New: subreg truncation not hoisted from loop

2015-05-21 Thread jon at beniston dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66248 Bug ID: 66248 Summary: subreg truncation not hoisted from loop Product: gcc Version: 5.1.0 Status: UNCONFIRMED Severity: enhancement Priority: P3 Component:

Re: Fix hashing of basetypes of methods

2015-05-21 Thread Richard Biener
On May 21, 2015 6:02:27 PM GMT+02:00, Jan Hubicka hubi...@ucw.cz wrote: Hi, this patch drops TYPE_METHOD_BASETYPE from hash_canonical_type. It is not compared by gimple_canonical_types_compatible_p and thus it can only corrupt the hashtable by having two entries that are equal but having

Re: Do not compare type attributes in gimple_canonical_types_compatible_p

2015-05-21 Thread Richard Biener
On May 21, 2015 6:06:18 PM GMT+02:00, Jan Hubicka hubi...@ucw.cz wrote: Hi, this patch removes call to comp_type_attributes (wich happens for METHOD_TYPE and FUNCTION_TYPE only). This does not make sense, because type attributes may change in variants and pointers should be considered compatible.

Re: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option

2015-05-21 Thread Joseph Myers
On Thu, 21 May 2015, Rainer Orth wrote: @@ -1864,6 +1873,12 @@ libgcc.mvars: config.status Makefile specs xgcc$(exeext) echo GCC_CFLAGS = '$(GCC_CFLAGS)' tmp-libgcc.mvars echo INHIBIT_LIBC_CFLAGS = '$(INHIBIT_LIBC_CFLAGS)' tmp-libgcc.mvars echo TARGET_SYSTEM_ROOT =

Avoid non-canonical RTL in splitter for adding a large constant to register on the PA

2015-05-21 Thread Jeff Law
The PA has a splitter to optimize the addition of certain constants to a register. One of the cases the splitter handles is when the constant requires 2 insns to generate, is divisible by 2, 4, or 8 and if divided by 2, 4 or 8 it only needs a single insn to generate the constant. Obviously

[Bug target/66224] PowerPC _GLIBCXX_READ_MEM_BARRIER too weak

2015-05-21 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66224 --- Comment #2 from David Edelsohn dje at gcc dot gnu.org --- Author: dje Date: Thu May 21 17:18:25 2015 New Revision: 223496 URL: https://gcc.gnu.org/viewcvs?rev=223496root=gccview=rev Log: PR target/66224 *

Re: [PATCH][AArch64] Add __extension__ and __always_inline__ to crypto intrinsics

2015-05-21 Thread James Greenhalgh
On Thu, May 21, 2015 at 03:42:33PM +0100, Kyrill Tkachov wrote: Hi all, The crypto intrinsics are missing an __extension__ and an __always_inline__ attribute that all the other intrinsics have. I don't see any reason for them to be different and the always_inline attribute will be needed if

[Bug c/66247] make check-gmp fails for gcc-5.1.0

2015-05-21 Thread mlmar...@clearsky-data.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66247 --- Comment #3 from Matthew L. Martin mlmar...@clearsky-data.com --- The source code in question is downloaded as a prerequisite for building gcc-5.1.0 (gmp). Where should I file a bug against the test?

[Bug c/65446] Improve -Wformat-signedness

2015-05-21 Thread eblake at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65446 --- Comment #4 from Eric Blake eblake at redhat dot com --- Arguably, %u with short should warn, while %hu with short should not. On the other hand, if I use %hu with int, it is unclear to me whether I should get a warning (the fact that I'm

Re: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option

2015-05-21 Thread H.J. Lu
On Thu, May 21, 2015 at 7:13 AM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: H.J. Lu hjl.to...@gmail.com writes: Here is the complete patch. Tested on Linux/x86-64. It is also available on hjl/pie/master branch in git mirror. As always, please keep generated files like configure and

[Bug c/66247] make check-gmp fails for gcc-5.1.0

2015-05-21 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66247 --- Comment #4 from Marek Polacek mpolacek at gcc dot gnu.org --- I think send a report to gmp-b...@gmplib.org.

[Bug c++/66239] Unoptimized sqrt(float or double) returns wrong values for ARM Cortex-A8 -mfloat-abi=[soft,softfp]

2015-05-21 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66239 Andrew Pinski pinskia at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |WAITING

<    1   2   3   4