Re: [PATCH 1/2] add unique_ptr header

On Fri, Aug 04, 2017 at 08:55:50PM +0100, Jonathan Wakely wrote: > On 01/08/17 23:09 -0400, Trevor Saunders wrote: > > aiui C++03 is C++98 with a few additions to the stl. > > Again, STL != C++ Standard Library. > > C++03 made some important changes to the core language, particularly > the

Re: [PATCH 0/2] add unique_ptr class

On Fri, Aug 04, 2017 at 07:52:18PM +0100, Jonathan Wakely wrote: > On 31/07/17 19:46 -0400, tbsaunde+...@tbsaunde.org wrote: > > I've been saying I'd do this for a long time, but I'm finally getting to > > importing the C++98 compatable unique_ptr class Pedro wrote for gdb a while > > back. I

Re: [PATCH 0/2] add unique_ptr class

On Fri, Aug 04, 2017 at 04:43:32PM -0600, Martin Sebor wrote: > On 07/31/2017 05:46 PM, tbsaunde+...@tbsaunde.org wrote: > > From: Trevor Saunders > > > > Hi, > > > > I've been saying I'd do this for a long time, but I'm finally getting to > > importing the C++98

[Bug fortran/81723] New: [7/8 Regression] fortran build doesn't terminate on 64bit targets

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81723 Bug ID: 81723 Summary: [7/8 Regression] fortran build doesn't terminate on 64bit targets Product: gcc Version: 7.1.1 Status: UNCONFIRMED Severity: normal

[Bug c++/81722] [7/8 Regression] memory hog building c++ on i686-linux-gnu

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81722 Matthias Klose changed: What|Removed |Added Target|i686-linux-gnu |i686-linux-gnu, |

Re: [PATCH 00/22] RFC: integrated 3rd-party static analysis support

On 8/4/17, David Malcolm wrote: > This patch kit clearly isn't ready yet as-is (see e.g. the > "known unknowns" below), but I'm posting it now in the hope of > getting early feedback. > > Summary > === > > This patch kit provides an easy way to make integrate 3rd-party

[Bug c++/81722] New: [7/8 Regression] memory hog building c++ on i686-linux-gnu

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81722 Bug ID: 81722 Summary: [7/8 Regression] memory hog building c++ on i686-linux-gnu Product: gcc Version: 7.1.1 Status: UNCONFIRMED Severity: normal

[Bug target/81490] x86: Handling of symbol ranges for __seg_fs/__seg_gs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81490 --- Comment #28 from Andy Lutomirski --- hjl, I don't understand what problem you're trying to solve here. Your patch makes it relatively straightforward to access global variables using __seg_gs if the environment sets up the %gs base exactly

[Bug target/56010] Powerpc, -mcpu=native and -mtune=native use the wrong name for target 7450

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56010 James Clarke changed: What|Removed |Added CC||glaubitz at physik dot fu-berlin.d

Re: [PATCH 0/2] add unique_ptr class

On 07/31/2017 05:46 PM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders Hi, I've been saying I'd do this for a long time, but I'm finally getting to importing the C++98 compatable unique_ptr class Pedro wrote for gdb a while back. I believe the gtl namespace

Re: [PATCH] Simplify pow with constant

On Fri, 4 Aug 2017, Richard Biener wrote: > >> Do this only for fast-math as accuracy is reduced. This is much faster > >> since pow is more complex than exp - with a current GLIBC the speedup > >> is more than 7 times for this transformation. > > > > Is it bound to be so on future glibc

Re: help with PR78809 - inline strcmp for small constant strings

On Fri, Aug 04, 2017 at 08:38:11PM +, Wilco Dijkstra wrote: > Richard Henderson wrote:  > > On 08/04/2017 05:59 AM, Prathamesh Kulkarni wrote: > > > > For i386, it seems strcmp is expanded inline via cmpstr optab by > > > expand_builtin_strcmp if one of the strings is constant. Could we

[Bug c++/79790] [C++17] ICE class template argument deduction failed

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79790 Paolo Carlini changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c++/79790] [C++17] ICE class template argument deduction failed

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79790 --- Comment #7 from paolo at gcc dot gnu.org --- Author: paolo Date: Fri Aug 4 22:15:48 2017 New Revision: 250883 URL: https://gcc.gnu.org/viewcvs?rev=250883=gcc=rev Log: /cp 2017-08-04 Paolo Carlini PR

[Bug c++/79790] [C++17] ICE class template argument deduction failed

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79790 --- Comment #6 from paolo at gcc dot gnu.org --- Author: paolo Date: Fri Aug 4 22:13:46 2017 New Revision: 250882 URL: https://gcc.gnu.org/viewcvs?rev=250882=gcc=rev Log: /cp 2017-08-04 Paolo Carlini PR

[Bug middle-end/81705] [8 Regression] UBSAN: yet another false positive

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81705 --- Comment #7 from Dmitry Babokin --- That's an excellent new! This means UBSAN becomes finally fully functional in GCC. No known false positives anymore (on quite large test base). Great job and thank you!

[PATCH 20/22] Add checkers/splint.py

This patch adds a harness for invoking splint: http://splint.org/ returning the results in JSON format. It runs "splint -csv TEMPFILE +csvoverwrite -strict", then uses firehose.parsers.splint.parse_splint_csv and firehose.parsers.splint.parse_splint_stderr to parse the csv and the stderr,

[PATCH 05/22] diagnostic.c/h: add support for external tools

This patch adds fields "external_tool" and "external_test_id" to diagnostic_info, allowing for diagnostics to be marked as coming from a 3rd-party tool. Instead of printing the pertinent warning flag e.g.: foo.c:10:1: something is wrong [-Wpointer-arith] the tool "ID" and (optionally) test ID

[PATCH 13/22] Add checkers/checker.py

This patch: * creates a new "checkers" top-level directory to hold harnesses for 3rd-party code-checking tools. * adds a "checker.py" Python module for use when implementing such harnesses 3rd-party code-checking tools are expected to be passed command-line arguments by the frontend, and to

[PATCH 18/22] Add checkers/flawfinder.py

This patch adds a harness for invoking flawfinder: https://www.dwheeler.com/flawfinder/ returning the results in JSON format. It runs "flawfinder", then uses firehose.parsers.flawfinder.parse_file to parse the stdout, turning it into firehose JSON. checkers/ChangeLog: * flawfinder.py:

[PATCH 21/22] Add checkers/Makefile

This patch adds a rather simplistic Makefile to "checkers", purely for exercising the test suites of the various harness. checkers/ChangeLog: * Makefile: New file. --- checkers/Makefile | 23 +++ 1 file changed, 23 insertions(+) create mode 100644 checkers/Makefile

[PATCH 12/22] Add -Wrun-analyzers= to common.opt, toplev.c, and invoke.texi

This patch introduces -Wrun-analyzers= and wires it up to the checkers.cc code, via toplev.c As noted earlier, I'm not a great fan of the monolithic option syntax here; maybe there should be some kind of option that can be supplied multiple times for adding individual checkers. gcc/ChangeLog:

[PATCH 14/22] Add checkers/always_fails.py

This patch adds a "checker" that always fails, for ensuring that we can handle failed runs of 3rd-party tools. checkers/ChangeLog: * always_fails.py: New file. --- checkers/always_fails.py | 57 1 file changed, 57 insertions(+) create

[PATCH 06/22] Makefile.in: hack in -lpthread

The checker.cc patch later in the kit can optionally make use of pthread if available. Doing it properly would involve some configure checks; this patch simply hacks in -lpthread into LIB unconditionally for now. gcc/ChangeLog: * Makefile.in (LIB): Hack in -lpthread. --- gcc/Makefile.in

[PATCH 15/22] Add checkers/clang_analyzer.py

This patch adds a harness for invoking clang's static analyzer: https://clang-analyzer.llvm.org/ returning the results in JSON format. It runs scan-build, then uses firehose.parsers.clanganalyzer.parse_plist to parse the generated .plist file, turning them into firehose JSON.

[PATCH 04/22] Add firehose.h/cc

"Firehose" is a serialization format for results from code analysis tools: http://firehose.readthedocs.io/en/latest/index.html (along with a Python module for working with the format). This patch implements a set of C++ classes modeling the format, with support for populating them from a JSON

[PATCH 16/22] Add checkers/coverity.py

This patch is an example of supporting a proprietary 3rd-party tool: a harness for invoking the Coverity checker. It uses the '--json-output-v2' option to cov-format-errors, and then uses firehose.parsers.coverity.parse_json_v2 to parse the generated Coverity JSON format, turning it into firehose

[PATCH 11/22] Add checkers/test-sources

This patch adds a collection of test sources for use by the testsuites of the various checker harnesses. checkers/ChangeLog: * test-sources/conditional-leak.c: New file. * test-sources/cpychecker-demo.c: New file. * test-sources/divide-by-zero.c: New file. *

[PATCH 09/22] Add selftest::read_file (..., FILE *, ...)

This patch is a hack used by the followup checkers.cc patch, and ought to be removed in any final version of the kit. gcc/ChangeLog: * selftest.c (read_file): New overload. * selftest.h (read_file): New overload. --- gcc/selftest.c | 16 +--- gcc/selftest.h | 7

[PATCH 22/22] Add contrib/get-static-analysis.py

This patch adds a simple Python 2/3 script for reading the static analysis "watermark" from object files, writing the JSON to stdout (prettyprinting it with indentation and newlines for ease of human reading). contrib/ChangeLog: * get-static-analysis.py: New file. ---

[PATCH 10/22] Add checkers.h/cc

This implements a new "policy" class to read a description of a set of checkers to run, along with a "checker" class to handle actually running the checkers, ouputting the results through gcc's diagnostic subsystem, and watermarking the generated binary with the results and metadata. Caveats: *

[PATCH 07/22] Add minimal version of Nick Clifton's annobin code

This patch provides a way to "watermark" binaries with metadata. It's used later in the patch kit to watermark binaries with static analysis results and metadata. See: https://fedoraproject.org/wiki/Toolchain/Watermark Note: this is a version of Nick Clifton's "annobin" gcc plugin:

[PATCH 19/22] Add checkers/ianal.py

This patch is a demo of handling code metrics and metadata ("info" results in Firehose terminology). It adds a standalone tool harness which scans the main input file looking for "Copyright" lines, returning information on them as firehose JSON. When sent through GCC's diagnostic subsystem by

[PATCH 08/22] Add GNU_BUILD_ATTRIBUTE_STATIC_ANALYSIS to annobin.h

gcc/ChangeLog: * annobin.h (GNU_BUILD_ATTRIBUTE_STATIC_ANALYSIS): New define. --- gcc/annobin.h | 1 + 1 file changed, 1 insertion(+) diff --git a/gcc/annobin.h b/gcc/annobin.h index 76eb01c..1152316 100644 --- a/gcc/annobin.h +++ b/gcc/annobin.h @@ -35,6 +35,7 @@ #define

[PATCH 17/22] Add checkers/cppcheck.py

This patch adds a harness for invoking cppcheck: http://cppcheck.sourceforge.net/ returning the results in JSON format. It runs "cppcheck --xml --xml-version=2", then uses firehose.parsers.cppcheck.parse_file to parse the generated .xml file, turning it into firehose JSON. checkers/ChangeLog:

[PATCH 03/22] Add JSON implementation

This patch adds support to gcc for reading and writing JSON, based on DOM-like trees of json::value instances. gcc/ChangeLog: * Makefile.in (OBJS): Add json.o. * json.cc: New file. * json.h: New file. * selftest-run-tests.c (selftest::run_tests): Call

[PATCH 00/22] RFC: integrated 3rd-party static analysis support

This patch kit clearly isn't ready yet as-is (see e.g. the "known unknowns" below), but I'm posting it now in the hope of getting early feedback. Summary === This patch kit provides an easy way to make integrate 3rd-party static analysis tools into gcc, and have them: (a) report through

[PATCH 01/22] Expose assert_loceq outside of input.c; add ASSERT_LOCEQ

gcc/ChangeLog: * input.c: Include "selftest-input.h". (selftest::assert_loceq): Remove "static". Add "report_loc" param and update assertions to use it. (selftest::test_accessing_ordinary_linemaps): Use ASSERT_LOCEQ rather than assert_loceq.

[PATCH 02/22] libcpp: add linemap_position_for_file_line_and_column

gcc/ChangeLog: * input.c (selftest::test_making_arbitrary_locations): New function. (selftest::input_c_tests): Call it. libcpp/ChangeLog: * include/line-map.h (linemap_position_for_file_line_and_column): New decl. * line-map.c

[Bug c++/69855] Missing diagnostic for overload that only differs by return type

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69855 Martin Sebor changed: What|Removed |Added CC||msebor at gcc dot gnu.org --- Comment #7

[Bug c++/81608] incompatible declarations of the same extern function at different scopes accepted

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81608 Martin Sebor changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug target/81490] x86: Handling of symbol ranges for __seg_fs/__seg_gs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81490 --- Comment #27 from H.J. Lu --- (In reply to H. Peter Anvin from comment #26) > @GPREL (altough it probably should be @GPOFF by analogy with @TPOFF?) gives > the linker an option to distinguish the relocations which need to be > adjusted at

Re: help with PR78809 - inline strcmp for small constant strings

On 08/04/2017 01:38 PM, Wilco Dijkstra wrote: >> For constant strings of small length (upto 3?), I was wondering if it'd be a >> good idea to manually unroll strcmp loop, similar to __strcmp_* macros in >> bits/string.h?> >> For eg in gimple-fold, transform >> x = __builtin_strcmp(s, "ab") >> to

Re: help with PR78809 - inline strcmp for small constant strings

Richard Henderson wrote:  > On 08/04/2017 05:59 AM, Prathamesh Kulkarni wrote: > > For i386, it seems strcmp is expanded inline via cmpstr optab by > > expand_builtin_strcmp if one of the strings is constant. Could we similarly > > define cmpstr pattern for AArch64? > > Certainly that's

[PATCH][PING][PR target/81535] Fix tests on Power

Hi all, This patch fixes issues reported in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81535 I removed call to g in pr79439.c because gcc was duplicating the basic block with call depending on compiler flags (so scan-assembler-times pattern wasn't reliable anymore). I also added alias to

[Bug tree-optimization/57371] Simplify (double)i != 0

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57371 --- Comment #5 from Yury Gribov --- Author: ygribov Date: Fri Aug 4 20:29:12 2017 New Revision: 250877 URL: https://gcc.gnu.org/viewcvs?rev=250877=gcc=rev Log: Remove useless floating point casts in comparisons. 2017-08-04 Yury Gribov

[Bug c++/81634] Some types are incorrectly detected as not standard layout

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81634 Jonathan Wakely changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug c++/81608] incompatible declarations of the same extern function at different scopes accepted

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81608 --- Comment #2 from Jonathan Wakely --- See PR 69855 comment 2 - is this a dup?

[Bug libstdc++/81706] std::sin vectorization bug

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #8 from Jonathan Wakely --- (In reply to Richard Biener from comment #1) > The C library sin is vectorized by being appropriately annotated with OpenMP > SIMD attributes. This isn't the case for the libstdc++ variant which ends > up

[Bug libstdc++/81482] by-value lambda capture in remove_if

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81482 --- Comment #3 from Jonathan Wakely --- (In reply to kloetzl from comment #2) > I don't think that the runtime cost of the copies is measurable. My bigger > issue is that this quirks exposes the internal behaviour of the algorithm >

[Bug c++/81674] gcc cannot detect missing initialisers for fields in constructors

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81674 --- Comment #1 from Jonathan Wakely --- There are several bugs about this already, please search for duplicates.

Re: [PATCH 1/2] add unique_ptr header

On 01/08/17 23:09 -0400, Trevor Saunders wrote: aiui C++03 is C++98 with a few additions to the stl. Again, STL != C++ Standard Library. C++03 made some important changes to the core language, particularly the value-initialization rules. Most of the library changes were small bug fixes, and

[Bug c++/81660] Add -Wexceptions warning

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81660 Jonathan Wakely changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug target/81490] x86: Handling of symbol ranges for __seg_fs/__seg_gs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81490 --- Comment #26 from H. Peter Anvin --- @GPREL (altough it probably should be @GPOFF by analogy with @TPOFF?) gives the linker an option to distinguish the relocations which need to be adjusted at link/load time and the ones that don't. We have

Re: [patch, gcc doc + fortran] Make -Ofast honor -fmax-stack-var-size

Am 04.08.2017 um 14:09 schrieb Thomas Koenig: Hello world, the attached patch This time, really attached, even with ChangeLog! Regards Thomas 2017-08-04 Thomas Koenig PR fortran/68829 * doc/invoke.texi: Document change in behvaior for -Ofast

[Bug fortran/52387] I/O output of write after nonadvancing read

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52387 Jerry DeLisle changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|WONTFIX

[Bug fortran/44292] [libgfortran ABI breakage] Increase internal size of RECL= of the OPEN statement

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=44292 Jerry DeLisle changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |jvdelisle at gcc dot gnu.org

Re: [PATCH 0/2] add unique_ptr class

On 31/07/17 19:46 -0400, tbsaunde+...@tbsaunde.org wrote: I've been saying I'd do this for a long time, but I'm finally getting to importing the C++98 compatable unique_ptr class Pedro wrote for gdb a while back. I believe the gtl namespace also comes from Pedro, but GNU template library seems

Re: x86 branches vs conditional moves

On Sat, Jul 8, 2017 at 1:30 AM, Michael Clark wrote: > Why does gcc branch on _Bool, but emits a conditional move for an integer? > can it emit cmovne instead of branching? also curious where one would change > this to learn about GCC internals. The RTL generated for the int and _Bool case is

Re: [PATCH] Fix switch expansion (PR middle-end/81698)

On August 4, 2017 5:42:21 PM GMT+02:00, Jakub Jelinek wrote: >Hi! > >As mentioned inthe PR, the switch expansion relies on EDGE_SUCC (bb, 0) >from a GIMPLE_SWITCH to be the default edge, but GIMPLE only ensures >that >default label is the first one. > >So, this patch (in

Re: help with PR78809 - inline strcmp for small constant strings

On 08/04/2017 05:59 AM, Prathamesh Kulkarni wrote: > Hi, > I was having a look at PR78809. > For the test-case: > int t1(const char *s) { return __builtin_strcmp (s, "a"); } > > for aarch64, trunk with -O2 generates: > t1: > adrpx1, .LANCHOR0 > add x1, x1, :lo12:.LANCHOR0

Re: [PATCH, rs6000] Move vec_mulo() and vec_mule() builtin test cases to P8 test file.

On Fri, Aug 04, 2017 at 10:27:29AM -0700, Carl Love wrote: > The following patch moves the vec_mule() and vec_mulo() builtin test > cases to the builtins-3-p8.c test case file. These builtins use P8 > instructions so they need to be in the P8 test file. Okay for trunk. Thanks for fixing this!

Re: [C++ Patch] PR 79790 ("[C++17] ICE class template argument deduction")

Hi, On 04/08/2017 20:05, Tim Song wrote: These messages are only emitted when elided == true, which, if I'm reading the code correctly, is only the case when you are in copy-initialization context (and have skipped at least one explicit guide). Ah! Thanks! Paolo.

Re: [PATCH 2/3] Matching tokens: C parts (v2)

On 08/04/2017 08:32 AM, David Malcolm wrote: > On Thu, 2017-08-03 at 11:34 -0600, Jeff Law wrote: >> On 08/01/2017 02:21 PM, David Malcolm wrote: >>> Changed in v2: >>> >>> * Renamed template argument to traits_t; eliminated subclasses, >>> just >>> using traits struct. >>> * Moved enum

Re: [C++ Patch] PR 79790 ("[C++17] ICE class template argument deduction")

On Fri, Aug 4, 2017 at 1:32 PM, Paolo Carlini wrote: > About the exact wording, I'm a little puzzled, because, besides the > "implicit" nit, the "copy-inizialization" already occurs in two other places > in that function, in the preceding: > > if (elided && !cands) >

Re: Upstreaming very old changes

On 08/04/2017 11:20 AM, co...@sdf.org wrote: > Hi, GCC! > > I believe netbsd is the primary user of the vax target. its status is: > > good: netbsd uses gcc 5.4.0, and cross compiles its userland+kernel with > this. it runs and is also able to natively build useful programs like perl. > > bad:

[Bug target/81590] AVX512 run-time test failures

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81590 H.J. Lu changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: [PATCH] Fix PR81354 (rewrite gimple_split_edge)

On Aug 1, 2017, at 10:56 AM, Bill Schmidt wrote: > > On Aug 1, 2017, at 8:50 AM, Bill Schmidt wrote: >> >> On Aug 1, 2017, at 7:44 AM, Bill Schmidt wrote: >>> On Aug 1, 2017, at 3:46 AM, Richard

[Bug target/81590] AVX512 run-time test failures

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81590 --- Comment #5 from hjl at gcc dot gnu.org --- Author: hjl Date: Fri Aug 4 17:48:57 2017 New Revision: 250875 URL: https://gcc.gnu.org/viewcvs?rev=250875=gcc=rev Log: i386: Rewrite check for AVX512 features Add a new file, avx512-check.h, to

[Bug target/81720] [arm] Invalid code generation

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81720 --- Comment #6 from Orlando Arias --- (In reply to Richard Earnshaw from comment #5) > Bare metal target compilers should still respect the standard, which is > quite clear that dereferencing NULL is undefined. Some users of the > compiler

[Bug target/81490] x86: Handling of symbol ranges for __seg_fs/__seg_gs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81490 --- Comment #25 from H.J. Lu --- Comment on attachment 41920 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41920 A binutils patch This has been replaced by users/hjl/gprel/master branch at

Re: [C++ Patch] PR 79790 ("[C++17] ICE class template argument deduction")

Hi, On 04/08/2017 19:00, Jason Merrill wrote: On 07/14/2017 01:32 PM, Paolo Carlini wrote: + error ("cannot deduce template arguments for copy-initialization" + " of %qT, as it has no viable implicit deduction guides", type); Why "copy-initialization"? We do deduction for

[PATCH, rs6000] Move vec_mulo() and vec_mule() builtin test cases to P8 test file.

GCC Maintainers: The following patch moves the vec_mule() and vec_mulo() builtin test cases to the builtins-3-p8.c test case file. These builtins use P8 instructions so they need to be in the P8 test file. Please let me know if the following patch is acceptable. Thanks.

[Bug target/81490] x86: Handling of symbol ranges for __seg_fs/__seg_gs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81490 --- Comment #24 from H.J. Lu --- (In reply to Andy Lutomirski from comment #20) > > Can someone elaborate on what the @GPREL suffix actually means? "%seg:foo@GPREL" is used to address symbol, foo, relative to __gp. foo is addressed by %seg +

Re: layout of __attribute__((packed)) vs. #pragma pack

On 07/28/2017 04:51 AM, Geza Herman wrote: There's an option in GCC "-mms-bitfields". The doc about it begins with: "If packed is used on a structure, or if bit-fields are used, it may be that the Microsoft ABI lays out the structure differently than the way GCC normally does. Particularly

Upstreaming very old changes

Hi, GCC! I believe netbsd is the primary user of the vax target. its status is: good: netbsd uses gcc 5.4.0, and cross compiles its userland+kernel with this. it runs and is also able to natively build useful programs like perl. bad: -O0 in places, text relocations. obvious signs of more bugs

[Bug target/81490] x86: Handling of symbol ranges for __seg_fs/__seg_gs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81490 H.J. Lu changed: What|Removed |Added Attachment #41920|0 |1 is obsolete|

Re: [C++ Patch] PR 79790 ("[C++17] ICE class template argument deduction")

On 07/14/2017 01:32 PM, Paolo Carlini wrote: + error ("cannot deduce template arguments for copy-initialization" +" of %qT, as it has no viable implicit deduction guides", type); Why "copy-initialization"? We do deduction for direct-initialization, as well. I would also

Re: [C++ committed] Fix ICE with -Wshadow=compatible-local (PR c++/81640)

On Fri, Aug 04, 2017 at 12:50:36PM -0400, Jason Merrill wrote: > On Wed, Aug 2, 2017 at 3:38 AM, Jakub Jelinek wrote: > > I've bootstrapped/regtested the following patch last night and committed > > after preapproval from Nathan, though of course a question is if > >

Re: [PATCH v2][RFC] Canonize names of attributes.

On Fri, Aug 4, 2017 at 9:42 AM, Martin Liška wrote: > On 08/02/2017 01:25 PM, Joseph Myers wrote: >> On Thu, 13 Jul 2017, Martin Liška wrote: >> >>> +/* For a given IDENTIFIER_NODE, strip leading and trailing '_' characters >>> + so that we have a canonical form of attribute

Re: [C++ committed] Fix ICE with -Wshadow=compatible-local (PR c++/81640)

On Wed, Aug 2, 2017 at 3:38 AM, Jakub Jelinek wrote: > I've bootstrapped/regtested the following patch last night and committed > after preapproval from Nathan, though of course a question is if > lookup_fnfields_slot/lookup_fnfields_slot_nolazy shouldn't have > if

[Bug c++/81721] New: precompiled header : internal compiler error: Segmentation fault

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81721 Bug ID: 81721 Summary: precompiled header : internal compiler error: Segmentation fault Product: gcc Version: 7.1.0 Status: UNCONFIRMED Severity: normal

[Bug preprocessor/45599] Remove all code applying to obsolete "#pragma once"

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=45599 Eric Gallager changed: What|Removed |Added Status|SUSPENDED |RESOLVED Resolution|---

[Bug target/81720] [arm] Invalid code generation

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81720 Richard Earnshaw changed: What|Removed |Added Resolution|INVALID |WONTFIX --- Comment #5 from Richard

[Bug c++/81702] [7/8 Regression] ICE in gimple_get_virt_method_for_vtable

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81702 Martin Jambor changed: What|Removed |Added CC||hubicka at gcc dot gnu.org --- Comment

[Bug target/81720] [arm] Invalid code generation

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81720 Andrew Pinski changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug target/81720] [arm] Invalid code generation

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81720 Orlando Arias changed: What|Removed |Added Status|RESOLVED|UNCONFIRMED Resolution|INVALID

Re: [PING^4][PATCH v2] Generate reproducible output independently of the build-path

On Fri, Aug 4, 2017 at 1:32 PM, Matthias Klose wrote: > On 03.08.2017 12:05, Jeff Law wrote: >> On 08/02/2017 08:06 PM, Ximin Luo wrote: >>> Jeff Law: On 07/21/2017 10:15 AM, Ximin Luo wrote: > (Please keep me on CC, I am not subscribed) > > > Proposal >

[Bug target/81720] [arm] Invalid code generation

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81720 Andrew Pinski changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug target/81720] [arm] Invalid code generation

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81720 --- Comment #1 from Orlando Arias --- Created attachment 41929 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41929=edit Compiler output with erroneous assembly listing.

[Bug target/81720] New: [arm] Invalid code generation

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81720 Bug ID: 81720 Summary: [arm] Invalid code generation Product: gcc Version: 7.1.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: target

[Bug hsa/81713] BIT_FIELD_REF produced by Brig FE do not pass new verification

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81713 --- Comment #4 from visit0r at gcc dot gnu.org --- Author: visit0r Date: Fri Aug 4 15:50:14 2017 New Revision: 250874 URL: https://gcc.gnu.org/viewcvs?rev=250874=gcc=rev Log: Fix PR 81713 * brigfrontend/brig-basic-inst-handler.cc: replace

[Bug hsa/81713] BIT_FIELD_REF produced by Brig FE do not pass new verification

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81713 --- Comment #3 from Pekka Jääskeläinen --- Committed in r250874.

[Bug c++/81719] New: Range-based for loop on short fixed size array generates long unrolled loop

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81719 Bug ID: 81719 Summary: Range-based for loop on short fixed size array generates long unrolled loop Product: gcc Version: 7.1.0 Status: UNCONFIRMED Severity:

[PATCH][AArch64] PR60580: Fix frame pointer option magic

To fix PR60580 simplify the logic in aarch64_override_options_after_change_1 (). If the frame pointer is enabled, set it to a special value that behaves similar to frame pointer omission. If we don't do this all leaf functions will get a frame pointer even if flag_omit_leaf_frame_pointer is

[PATCH] Fix switch expansion (PR middle-end/81698)

Hi! As mentioned inthe PR, the switch expansion relies on EDGE_SUCC (bb, 0) from a GIMPLE_SWITCH to be the default edge, but GIMPLE only ensures that default label is the first one. So, this patch (in expand_case) instead finds the default_edge from the default label and corresponding block.

Re: [PATCH] i386: Rewrite check for AVX512 features

On Fri, Aug 4, 2017 at 12:19 PM, Peryt, Sebastian wrote: >> -Original Message- >> From: Uros Bizjak [mailto:ubiz...@gmail.com] >> Sent: Sunday, July 30, 2017 11:02 AM >> To: H.J. Lu >> Cc: gcc-patches@gcc.gnu.org; Koval, Julia

[PATCH] Fix OpenMP/OpenACC/Cilk+ issues with forced/non-local labels (PR c/81687)

Hi! If there are forced or non-local labels in OpenMP parallel/task/target regions (or OpenACC or Cilk+), then we often fail to link as the following testcases show - the labels aren't emitted anywhere, just references to them. While it is true that for forced/non-local labels we can't clone the

Re: [PATCH] Simplify pow with constant

Richard Biener wrote: > On Fri, Aug 4, 2017 at 2:26 PM, Alexander Monakov wrote: > > On Fri, 4 Aug 2017, Wilco Dijkstra wrote: > >> This patch simplifies pow (C, x) into exp (x * C1), where C1 = log (C). > > > > I don't think you can do that for non-positive C. True, that can

[Bug preprocessor/45599] Remove all code applying to obsolete "#pragma once"

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=45599 --- Comment #6 from Jonathan Wakely --- I'd just go for RESOLVED INVALID. We don't want to remove the code that implements a supported feature.

  1   2   3   >