Re: [PATCH, V2 2/3] targhooks: New target hook for CTF/BTF debug info emission

2021-08-18 Thread Richard Biener via Gcc-patches
On Tue, Aug 17, 2021 at 7:26 PM Indu Bhagat wrote: > > On 8/17/21 1:04 AM, Richard Biener wrote: > > On Mon, Aug 16, 2021 at 7:39 PM Indu Bhagat wrote: > >> > >> On 8/10/21 4:54 AM, Richard Biener wrote: > >>> On Thu, Aug 5, 2021 at 2:52 AM Indu Bhagat via Gcc-patches > >>> wrote: > >

Re: Better memory statistics, take 2

2021-08-18 Thread Richard Biener via Gcc-patches
On Tue, Aug 17, 2021 at 9:31 PM Thomas Schwinge wrote: > > Hi! > > On 2021-08-17T09:27:46-0400, David Malcolm via Gcc-patches > wrote: > > On Tue, 2021-08-17 at 11:17 +0200, Thomas Schwinge wrote: > >> "Turn > >> global 'ggc_force_collect' variable into 'force_collect' parameter to > >> 'ggc_col

Re: [patch][version 6] add -ftrivial-auto-var-init and variable attribute "uninitialized" to gcc

2021-08-18 Thread Richard Biener via Gcc-patches
On Tue, 17 Aug 2021, Qing Zhao wrote: > > > > On Aug 17, 2021, at 9:50 AM, Qing Zhao via Gcc-patches > > wrote: > > > > > > > >> On Aug 17, 2021, at 3:29 AM, Richard Biener wrote: > >> > >> On Mon, 16 Aug 2021, Qing Zhao wrote: > >> > >>> My current code for expand_DEFERRED_INIT is like

Re: [patch][version 6] add -ftrivial-auto-var-init and variable attribute "uninitialized" to gcc

2021-08-18 Thread Richard Biener via Gcc-patches
On Tue, 17 Aug 2021, Qing Zhao wrote: > > > > On Aug 17, 2021, at 10:04 AM, Qing Zhao via Gcc-patches > > wrote: > > > > > > > >> On Aug 16, 2021, at 11:48 AM, Qing Zhao via Gcc-patches > >> wrote: > >> > From the above IR file after “FRE”, we can see that the major issue with > >>

Re: [PATCH] Adding target hook allows to reject initialization of register

2021-08-18 Thread Richard Biener via Gcc-patches
On Wed, Aug 18, 2021 at 3:54 AM Jojo R wrote: > > > — Jojo > 在 2021年8月16日 +0800 PM3:15,Richard Biener ,写道: > > On Fri, Aug 13, 2021 at 3:59 AM Jojo R wrote: > > > > — Jojo > 在 2021年8月11日 +0800 PM6:44,Richard Biener ,写道: > > On Wed, Aug 11, 2021 at 11:28 AM Richard Sandiford > wrote: > > > Richar

[committed] openmp: Actually ignore pragma_stmt pragmas for which c_parser_pragma returns false

2021-08-18 Thread Jakub Jelinek via Gcc-patches
Hi! Unlike the C++ FE, the C FE ignored pragmas (as if they weren't there) in pragma_stmt contexts if c*_parser_pragma returns false only when after labels, not inside of substatements of selection or loop statements. After making just that change, several gomp/goacc testcases started failing, bec

Re: [PATCH] move x86 to use gather/scatter internal functions

2021-08-18 Thread Richard Biener via Gcc-patches
On Wed, 18 Aug 2021, Hongtao Liu wrote: > On Wed, Aug 18, 2021 at 11:24 AM Hongtao Liu wrote: > > > > On Tue, Aug 17, 2021 at 10:43 PM Richard Biener via Gcc-patches > > wrote: > > > > > > On Tue, Aug 17, 2021 at 3:29 PM Richard Biener via Gcc-patches > > > wrote: > > > > > > > > This is an att

Re: [COMMITTED 3/3] Add GORI tracing faciltiies.

2021-08-18 Thread Aldy Hernandez via Gcc-patches
On 8/18/21 1:31 AM, Andrew MacLeod wrote: And this final patch provides tracing in the GORI component. This is what I used to find the ABS problem with https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101938 The code sequence looked like:     :     a1_8 = -arg1_7(D);     _1 = ABS_EXPR ;

Re: [PATCH] commit-mklog: Add --co argument.

2021-08-18 Thread Martin Liška
On 8/17/21 3:34 PM, Martin Liška wrote: On 8/17/21 3:09 PM, Martin Liška wrote: On 8/17/21 2:59 PM, Martin Liška wrote: with --trailer='Signed-off-by=Mona Lisa Octocat '. This should be of course: --trailer='Co-Authored-By=Mona Lisa Octocat ' Update version where mklog.py skips Co-Authored-

Re: [PATCH] inline: do not inline when no_profile_instrument_function is different

2021-08-18 Thread Martin Liška
On 6/23/21 3:15 PM, Martin Liška wrote: On 6/23/21 2:38 PM, Jan Hubicka wrote: Is there reason to prevent the inlining once instrumentation is done? No ;) I think you can just block it for early inliner. Sure. Do you have a handy predicate function that tells if einliner is done? Thanks,

[Patch] Fortran/OpenMP: Add memory routines existing for C/C++

2021-08-18 Thread Tobias Burnus
The added routines existed before for C/C++ (being part of OpenMP 5.0) but not for Fortran (new there since OpenMP 5.1) – as those are all bind(C), it only affects 'omp_lib' and uses the C interface otherwise. Note 1: OpenMP 5.1 added additional (target) memory routines for C/C++ and Fortran; tho

Re: [Patch] Fortran/OpenMP: Add memory routines existing for C/C++

2021-08-18 Thread Jakub Jelinek via Gcc-patches
On Wed, Aug 18, 2021 at 11:00:47AM +0200, Tobias Burnus wrote: > Note 2 --- > 'omp_lib.h' is included in as declaration-construct in the > specification-part of a file (i.e. possibly after an implicit statement); > hence, it cannot contain a use-stmt. Additiona

Re: [PATCH] move x86 to use gather/scatter internal functions

2021-08-18 Thread Hongtao Liu via Gcc-patches
On Wed, Aug 18, 2021 at 4:32 PM Richard Biener wrote: > > On Wed, 18 Aug 2021, Hongtao Liu wrote: > > > On Wed, Aug 18, 2021 at 11:24 AM Hongtao Liu wrote: > > > > > > On Tue, Aug 17, 2021 at 10:43 PM Richard Biener via Gcc-patches > > > wrote: > > > > > > > > On Tue, Aug 17, 2021 at 3:29 PM Ric

[committed] openmp: Add nothing directive support

2021-08-18 Thread Jakub Jelinek via Gcc-patches
Hi! As has been clarified, it is intentional that nothing directive is accepted in substatements of selection and looping statements and after labels and is handled as if the directive just isn't there, so that void foo (int x) { if (x) #pragma omp metadirective when (...:nothing) when (...:

Re: [PATCH] move x86 to use gather/scatter internal functions

2021-08-18 Thread Richard Biener via Gcc-patches
On Wed, 18 Aug 2021, Hongtao Liu wrote: > On Wed, Aug 18, 2021 at 4:32 PM Richard Biener wrote: > > > > On Wed, 18 Aug 2021, Hongtao Liu wrote: > > > > > On Wed, Aug 18, 2021 at 11:24 AM Hongtao Liu wrote: > > > > > > > > On Tue, Aug 17, 2021 at 10:43 PM Richard Biener via Gcc-patches > > > > w

Re: [PATCH] move x86 to use gather/scatter internal functions

2021-08-18 Thread Richard Biener via Gcc-patches
So in the end I seem to be able to combine AVX & AVX512 arriving at the following which passes basic testing. I will now see to teach the vectorizer the required "promotion" to handle mask_gather_loadv4dfv4si and mask_gather_loadv4sfv4di. Meanwhile, do you see any hole in the below? If not I'l

Re: [PATCH 3/3] aarch64: Remove macros for vld4[q]_lane Neon intrinsics

2021-08-18 Thread Christophe Lyon via Gcc-patches
On Mon, Aug 16, 2021 at 6:38 PM Jonathan Wright via Gcc-patches < gcc-patches@gcc.gnu.org> wrote: > Hi, > > This patch removes macros for vld4[q]_lane Neon intrinsics. This is a > preparatory step before adding new modes for structures of Advanced > SIMD vectors. > > Regression tested and bootstra

Re: [PATCH] move x86 to use gather/scatter internal functions

2021-08-18 Thread Hongtao Liu via Gcc-patches
On Wed, Aug 18, 2021 at 5:54 PM Richard Biener wrote: > > > So in the end I seem to be able to combine AVX & AVX512 arriving > at the following which passes basic testing. I will now see to > teach the vectorizer the required "promotion" to handle > mask_gather_loadv4dfv4si and mask_gather_loadv4

Re: [PATCH] PR fortran/100950 - ICE in output_constructor_regular_field, at varasm.c:5514

2021-08-18 Thread Tobias Burnus
Hi Harald, sorry for the belated review. On 03.08.21 23:17, Harald Anlauf wrote: allocate(x%str2, source="abd") if (len (x%str)) /= 1) ... if (len (x%str2(1:2) /= 2) ... etc. Namely: Search the last_ref = expr->ref->next->next ...? and then check that lastref? The mentioned search is now impl

Re: [PATCH] move x86 to use gather/scatter internal functions

2021-08-18 Thread Richard Biener via Gcc-patches
On Wed, 18 Aug 2021, Richard Biener wrote: > > So in the end I seem to be able to combine AVX & AVX512 arriving > at the following which passes basic testing. I will now see to > teach the vectorizer the required "promotion" to handle > mask_gather_loadv4dfv4si and mask_gather_loadv4sfv4di. > >

Re: [PATCH] move x86 to use gather/scatter internal functions

2021-08-18 Thread Richard Biener via Gcc-patches
On Wed, 18 Aug 2021, Hongtao Liu wrote: > On Wed, Aug 18, 2021 at 5:54 PM Richard Biener wrote: > > > > > > So in the end I seem to be able to combine AVX & AVX512 arriving > > at the following which passes basic testing. I will now see to > > teach the vectorizer the required "promotion" to han

(Re: [committed] openmp: Add nothing directive support)

2021-08-18 Thread Tobias Burnus
On 18.08.21 11:18, Jakub Jelinek wrote: As has been clarified, it is intentional that nothing directive is accepted in substatements of selection and looping statements and after labels and is handled as if the directive just isn't there ... And here is the Fortran version; as ST_NONE is used,

Re: (Re: [committed] openmp: Add nothing directive support)

2021-08-18 Thread Jakub Jelinek via Gcc-patches
On Wed, Aug 18, 2021 at 01:10:12PM +0200, Tobias Burnus wrote: > I think otherwise the patch is boring - as boring as 'omp nothing' itself > (outside its use in metadirectives). Yeah, I think nobody sane will use it for anything but metadirective except in compiler testsuites. > Fortran: Add Open

Re: [PATCH] move x86 to use gather/scatter internal functions

2021-08-18 Thread Hongtao Liu via Gcc-patches
On Wed, Aug 18, 2021 at 6:28 PM Richard Biener wrote: > > On Wed, 18 Aug 2021, Richard Biener wrote: > > > > > So in the end I seem to be able to combine AVX & AVX512 arriving > > at the following which passes basic testing. I will now see to > > teach the vectorizer the required "promotion" to h

Re: [PATCH] move x86 to use gather/scatter internal functions

2021-08-18 Thread Richard Biener via Gcc-patches
On Wed, 18 Aug 2021, Richard Biener wrote: > On Wed, 18 Aug 2021, Richard Biener wrote: > > > > > So in the end I seem to be able to combine AVX & AVX512 arriving > > at the following which passes basic testing. I will now see to > > teach the vectorizer the required "promotion" to handle > > m

Re: [PATCH] move x86 to use gather/scatter internal functions

2021-08-18 Thread Hongtao Liu via Gcc-patches
On Wed, Aug 18, 2021 at 7:30 PM Hongtao Liu wrote: > > On Wed, Aug 18, 2021 at 6:28 PM Richard Biener wrote: > > > > On Wed, 18 Aug 2021, Richard Biener wrote: > > > > > > > > So in the end I seem to be able to combine AVX & AVX512 arriving > > > at the following which passes basic testing. I wi

Re: [PATCH] move x86 to use gather/scatter internal functions

2021-08-18 Thread Hongtao Liu via Gcc-patches
On Wed, Aug 18, 2021 at 7:37 PM Hongtao Liu wrote: > > On Wed, Aug 18, 2021 at 7:30 PM Hongtao Liu wrote: > > > > On Wed, Aug 18, 2021 at 6:28 PM Richard Biener wrote: > > > > > > On Wed, 18 Aug 2021, Richard Biener wrote: > > > > > > > > > > > So in the end I seem to be able to combine AVX & AV

Add more self-tests for 'hash_map' with Value type with non-trivial constructor/destructor (was: Expensive selftests)

2021-08-18 Thread Thomas Schwinge
Hi! On 2021-08-17T10:57:36+0200, Richard Biener wrote: > On Tue, Aug 17, 2021 at 8:40 AM Thomas Schwinge > wrote: >> On 2021-08-16T14:10:00-0600, Martin Sebor wrote: >> > On 8/16/21 6:44 AM, Thomas Schwinge wrote: >> >> [...], to document the current behavior, I propose to >> >> "Add more self

Re: [PATCH 3/3] aarch64: Remove macros for vld4[q]_lane Neon intrinsics

2021-08-18 Thread Andreas Schwab
I think this patch breaks bootstrap. In file included from ../../libcpp/lex.c:756: /opt/gcc/gcc-20210818/Build/prev-gcc/include/arm_neon.h: In function 'float32x2x4_t vld4_lane_f32(const float32_t*, float32x2x4_t, int)': /opt/gcc/gcc-20210818/Build/prev-gcc/include/arm_neon.h:21081

Re: [PATCH v2] libstdc++: improve documentation for bits/stl_function.h [PR51539]

2021-08-18 Thread Jonathan Wakely via Gcc-patches
On Tue, 17 Aug 2021 at 21:39, Krzysztof Żelechowski wrote: > > PR libstdc++/PR51539 > > ChangeLog > * libstdc++-v3/include/bits/stl_function.h: Improve documentation. > > diff --git a/libstdc++-v3/include/bits/stl_function.h b/libstdc++-v3/include/ > bits/stl_function.h > index 073018d522

Re: [PATCH v2] libstdc++: improve documentation for bits/stl_function.h [PR51539]

2021-08-18 Thread Jonathan Wakely via Gcc-patches
On Wed, 18 Aug 2021 at 13:49, Jonathan Wakely wrote: > > On Tue, 17 Aug 2021 at 21:39, Krzysztof Żelechowski > wrote: > >template > > struct greater : public binary_function<_Tp, _Tp, bool> > > { > > + /// Tests whether parameter 1 is greater (`operator>`) than > > parameter

[PATCH] aarch64: Fix float <-> int errors in vld4[q]_lane intrinsics

2021-08-18 Thread Jonathan Wright via Gcc-patches
diford Subject: Re: [PATCH 3/3] aarch64: Remove macros for vld4[q]_lane Neon intrinsics   I think this patch breaks bootstrap. In file included from ../../libcpp/lex.c:756: /opt/gcc/gcc-20210818/Build/prev-gcc/include/arm_neon.h: In function 'float32x2x4_t vld4_lane_f32(const float32_t

Re: Expensive selftests (was: 'hash_map>')

2021-08-18 Thread David Edelsohn via Gcc-patches
This causes a bootstrap failure for me. PR/101959 On Tue, Aug 17, 2021 at 5:00 AM Richard Biener via Gcc wrote: > > On Tue, Aug 17, 2021 at 8:40 AM Thomas Schwinge > wrote: > > > > Hi! > > > > On 2021-08-16T14:10:00-0600, Martin Sebor wrote: > > > On 8/16/21 6:44 AM, Thomas Schwinge wrote: >

Re: [PATCH] move x86 to use gather/scatter internal functions

2021-08-18 Thread Richard Biener via Gcc-patches
On Wed, 18 Aug 2021, Hongtao Liu wrote: > On Wed, Aug 18, 2021 at 7:37 PM Hongtao Liu wrote: > > > > On Wed, Aug 18, 2021 at 7:30 PM Hongtao Liu wrote: > > > > > > On Wed, Aug 18, 2021 at 6:28 PM Richard Biener wrote: > > > > > > > > On Wed, 18 Aug 2021, Richard Biener wrote: > > > > > > > > >

Re: [PATCH, V2 2/3] targhooks: New target hook for CTF/BTF debug info emission

2021-08-18 Thread Indu Bhagat via Gcc-patches
On 8/18/21 12:00 AM, Richard Biener wrote: On Tue, Aug 17, 2021 at 7:26 PM Indu Bhagat wrote: On 8/17/21 1:04 AM, Richard Biener wrote: On Mon, Aug 16, 2021 at 7:39 PM Indu Bhagat wrote: On 8/10/21 4:54 AM, Richard Biener wrote: On Thu, Aug 5, 2021 at 2:52 AM Indu Bhagat via Gcc-patches

Re: [PATCH] aarch64: Fix float <-> int errors in vld4[q]_lane intrinsics

2021-08-18 Thread Richard Sandiford via Gcc-patches
t; > Subject: Re: [PATCH 3/3] aarch64: Remove macros for vld4[q]_lane Neon > intrinsics > > I think this patch breaks bootstrap. > > In file included from ../../libcpp/lex.c:756: > /opt/gcc/gcc-20210818/Build/prev-gcc/include/arm_neon.h: In function > 'float32x2x4_t vld

[committed] libstdc++: Fix vector printer tests to work in debug mode

2021-08-18 Thread Jonathan Wakely via Gcc-patches
This fixes a compilation error in debug mode, due to std::_Bit_reference not being defined, because it's in namespace std::__cxx1998 instead. We can refer to it as vector::reference instead, which always works. That fixes some compilation errors in debug mode, but the tests fail at run-time instea

[committed] libstdc++: Fix CTAD for debug sequence containers

2021-08-18 Thread Jonathan Wakely via Gcc-patches
This fixes some 23_containers/*/cons/deduction.cc failures seen with -std=c++17/-D_GLIBCXX_DEBUG, caused by non-immediate errors when substituting template arguments into an incorrect specialization of the std::__cxx1998 base class. This happens because the size_type member of the debug container i

Re: [patch][version 6] add -ftrivial-auto-var-init and variable attribute "uninitialized" to gcc

2021-08-18 Thread Qing Zhao via Gcc-patches
> On Aug 18, 2021, at 2:19 AM, Richard Biener wrote: > > On Tue, 17 Aug 2021, Qing Zhao wrote: > >> >> >>> On Aug 17, 2021, at 10:04 AM, Qing Zhao via Gcc-patches >>> wrote: >>> >>> >>> On Aug 16, 2021, at 11:48 AM, Qing Zhao via Gcc-patches wrote: >> From the abov

[committed] libstdc++: Simplify n-ary arithmetic promotion traits

2021-08-18 Thread Jonathan Wakely via Gcc-patches
The std::complex partial specializations have been unnecessary since 774c3d8647cc7012937cfc9d2d6dacc85b6cf8e9 Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: * include/ext/type_traits.h (__promote_2, __promote_3) (__promote_4): Redfine as alias templates using __promoted_

[committed] libstdc++: Enable doxygen processing for C++20 components

2021-08-18 Thread Jonathan Wakely via Gcc-patches
Improve grouping, add @since and @deprecated information. Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: * doc/doxygen/user.cfg.in (PREDEFINED): Enable doxygen processing for C++20 components and components that depend on compiler features. * include/bits

[committed] libstdc++: Improve doxygen comments in

2021-08-18 Thread Jonathan Wakely via Gcc-patches
Add notes about deprecation and modern replacements. Fix bogus "memory_adaptors" group name. Use markdown for formatting. Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: * include/bits/stl_function.h: Improve doxygen comments. Tested powerpc64le-linux. Committed to trunk. commi

[committed] libstdc++: Improve doxygen documentation for std::unique_ptr

2021-08-18 Thread Jonathan Wakely via Gcc-patches
Add more detailed documentation for unique_ptr and related components. The new alias templates for the _MakeUniq SFINAE helper make the generated docs look better too. Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: * include/bits/unique_ptr.h (default_delete): Add @since tag.

Re: [PATCH] move x86 to use gather/scatter internal functions

2021-08-18 Thread Richard Sandiford via Gcc-patches
Richard Biener writes: > On Wed, 18 Aug 2021, Hongtao Liu wrote: > >> On Wed, Aug 18, 2021 at 7:37 PM Hongtao Liu wrote: >> > >> > On Wed, Aug 18, 2021 at 7:30 PM Hongtao Liu wrote: >> > > >> > > On Wed, Aug 18, 2021 at 6:28 PM Richard Biener wrote: >> > > > >> > > > On Wed, 18 Aug 2021, Richar

Make 'gcc/hash-map-tests.c:test_map_of_type_with_ctor_and_dtor_expand' work on 32-bit architectures [PR101959]

2021-08-18 Thread Thomas Schwinge
Hi! On 2021-08-18T09:35:25-0400, David Edelsohn wrote: > This causes a bootstrap failure for me. > > PR/101959 Sorry for that; "details". ;-| OK to push the attached "Make 'gcc/hash-map-tests.c:test_map_of_type_with_ctor_and_dtor_expand' work on 32-bit architectures [PR101959]", which does res

Re: [patch][version 6] add -ftrivial-auto-var-init and variable attribute "uninitialized" to gcc

2021-08-18 Thread Qing Zhao via Gcc-patches
> On Aug 18, 2021, at 2:15 AM, Richard Biener wrote: > > On Tue, 17 Aug 2021, Qing Zhao wrote: > >> >> >>> On Aug 17, 2021, at 9:50 AM, Qing Zhao via Gcc-patches >>> wrote: >>> >>> >>> On Aug 17, 2021, at 3:29 AM, Richard Biener wrote: On Mon, 16 Aug 2021, Qing Zhao wro

Re: Make 'gcc/hash-map-tests.c:test_map_of_type_with_ctor_and_dtor_expand' work on 32-bit architectures [PR101959]

2021-08-18 Thread Richard Biener via Gcc-patches
On August 18, 2021 5:34:28 PM GMT+02:00, Thomas Schwinge wrote: >Hi! > >On 2021-08-18T09:35:25-0400, David Edelsohn wrote: >> This causes a bootstrap failure for me. >> >> PR/101959 > >Sorry for that; "details". ;-| OK to push the attached >"Make 'gcc/hash-map-tests.c:test_map_of_type_with_cto

Re: [PATCH 1/2] analyzer: detect and analyze calls via function pointer (GSoC)

2021-08-18 Thread David Malcolm via Gcc-patches
On Tue, 2021-08-17 at 20:40 +0530, Ankur Saini wrote: > Here is the final patch after fixing all the nits mentioned. > > Successfully bootstrapped and completed regress tests on x86_64-linux- > gnu. Great, thanks! A very minor nit re ChangeLog entries: when referencing a PR in a patch, typicall

Re: [PATCH v2] libstdc++: improve documentation for bits/stl_function.h [PR51539]

2021-08-18 Thread Jonathan Wakely via Gcc-patches
On Wed, 18 Aug 2021 at 13:49, Jonathan Wakely wrote: > > On Tue, 17 Aug 2021 at 21:39, Krzysztof Żelechowski wrote: > > - /// One of the @link memory_adaptors adaptors for member > > "memory_adaptors" here is a bad name (see > which introduced it) but we can just > ch

[pushed] Darwin: Handle the -rpath command line option.

2021-08-18 Thread Iain Sandoe
Hi, This handles the command line '-rpath' option by passing it through to the static linker. tested on x86_64, i686-darwin, pushed to master, thanks, Iain Signed-off-by: Iain Sandoe gcc/ChangeLog: * config.gcc: Include rpath.opt for Darwin. * config/darwin.h (DRIVER_SELF_SPEC

[pushed] libiberty, Darwin : Fix simple-object LTO table for cross-endian case.

2021-08-18 Thread Iain Sandoe
Hi, For mach-o, we encapsulate streamed IR for LTO in three special sections with a table that describes their entries. The table is expected to be written with native endianness for the target, but for cross-endian cross-compilation the swapping was omitted. Fixed thus. tested on a cross from

[pushed] Objective-C: fix crash with -fobjc-nilcheck

2021-08-18 Thread Iain Sandoe
Hi, Thanks to Matt for the PR and the initial draft patch. = When -fobjc-nilcheck is enabled, messages that result in a struct type should yield a zero-initialized struct when sent to nil. Currently, the frontend crashes when it encounters this situation. This patch fixes the crash by gen

[pushed] configure: Allow host fragments to react to --enable-host-shared.

2021-08-18 Thread Iain Sandoe
Hi, It is possible that a host makefile fragment might need to adapt to the setting of —enable-host-shared. This makes the host_shared value available to host makefile fragments. It uses this to adjust Darwin's mdynamic-no-pic in the case that shared host resources are required. tested on i686,

[pushed] Darwin, jit: Fix build [PR100613].

2021-08-18 Thread Iain Sandoe
Hi, The generic unix build is not completely suitable for Darwin platforms: * It is a convention to encode the library versioning in the binary and to have only one level of symlink for the installed files. This needs to be applied to the installation too. * The library needs to be built

Re: [PATCH 1/6] rs6000: Support SSE4.1 "round" intrinsics

2021-08-18 Thread Bill Schmidt via Gcc-patches
Hi, Paul! On 8/9/21 3:23 PM, Paul A. Clarke via Gcc-patches wrote: Suppress exceptions (when specified), by saving, manipulating, and restoring the FPSCR. Similarly, save, set, and restore the floating-point rounding mode when required. No attempt is made to optimize writing the FPSCR (by chec

Re: [PATCH 2/6] rs6000: Support SSE4.1 "min" and "max" intrinsics

2021-08-18 Thread Bill Schmidt via Gcc-patches
Hi Paul, On 8/9/21 3:23 PM, Paul A. Clarke via Gcc-patches wrote: Also, copy tests for _mm_min_epi8, _mm_min_epu16, _mm_min_epi32, _mm_min_epu32, _mm_max_epi8, _mm_max_epu16, _mm_max_epi32, _mm_max_epu32 from gcc/testsuite/gcc.target/i386. sse4_1-pmaxsb.c and sse4_1-pminsb.c were modified from

Re: [PATCH 3/6] rs6000: Simplify some SSE4.1 "test" intrinsics

2021-08-18 Thread Bill Schmidt via Gcc-patches
Hi Paul, On 8/9/21 3:23 PM, Paul A. Clarke via Gcc-patches wrote: Copy some simple redirections from i386 , for: - _mm_test_all_zeros - _mm_test_all_ones - _mm_test_mix_ones_zeros Testing, backports, etc. Simple is good!  LGTM. Bill 2021-08-09 Paul A. Clarke gcc * config/rs60

[PATCH] c++tools : Add a simple handler for ModuleCompiledRequest.

2021-08-18 Thread Iain Sandoe
Hi, I have found it useful when working with modules stuff to have the completed set of command/responses available (some people working with the interfaces for more sophisticated tools are using them). This message is a hand-shake telling the server that a CMI has been built, and for the simplis

Re: [PATCH 4/6] rs6000: Support SSE4.1 "cvt" intrinsics

2021-08-18 Thread Bill Schmidt via Gcc-patches
Hi Paul, On 8/9/21 3:23 PM, Paul A. Clarke via Gcc-patches wrote: Also, copy tests for: - _mm_cvtepi8_epi16, _mm_cvtepi8_epi32, _mm_cvtepi8_epi64 - _mm_cvtepi16_epi32, _mm_cvtepi16_epi64 - _mm_cvtepi32_epi64, - _mm_cvtepu8_epi16, _mm_cvtepu8_epi32, _mm_cvtepu8_epi64 - _mm_cvtepu16_epi32, _mm_cvt

Re: [PATCH 5/6] rs6000: Support more SSE4.1 "cmp", "mul", "pack" intrinsics

2021-08-18 Thread Bill Schmidt via Gcc-patches
Hi Paul, On 8/9/21 3:23 PM, Paul A. Clarke via Gcc-patches wrote: Also, copy tests for: - _mm_cmpeq_epi64, _mm_cmpgt_epi64 - _mm_mullo_epi32, _mm_mul_epi32 - _mm_packus_epi32 from gcc/testsuite/gcc.target/i386. Testing, backports, etc. This patch LGTM with the usual comment about documentin

Re: [PATCH 6/6] rs6000: Guard some x86 intrinsics implementations

2021-08-18 Thread Bill Schmidt via Gcc-patches
Hi Paul, On 8/9/21 3:23 PM, Paul A. Clarke via Gcc-patches wrote: Some compatibility implementations of x86 intrinsics include Power intrinsics which require POWER8. Guard them. emmintrin.h: - _mm_cmpord_pd: Remove code which was ostensibly for pre-POWER8, but which indeed depended on POWER

[PATCH] JIT, testsuite, Darwin: Initial testsuite fixes.

2021-08-18 Thread Iain Sandoe
Hi, * Note, the strategy in jit.exp has the assumption that $target==$host the patches here adhere to that - there is far less testsuite library support for host-side facilities (which we’d probably want to add if it was desirable to operate the Jit in a cross-compiler environment). ---

[PATCH] testsuite, JIT, Darwin: Adjust asm tests for Mach-O.

2021-08-18 Thread Iain Sandoe
Hi, This provides adjusted assembler fragments that are suitable for x86_64 Mach-O. ** as an aside, I wonder if it would be possible to configure some conditional on __ELF__, __MACH__, and CPU arch to pull in some object-cpu.inc so that we can generalize this for all hosts. this version te

Re: [PATCH] JIT, testsuite, Darwin: Initial testsuite fixes.

2021-08-18 Thread David Malcolm via Gcc-patches
On Wed, 2021-08-18 at 20:40 +0100, Iain Sandoe wrote: > Hi, > > * Note, the strategy in jit.exp has the assumption that > $target==$host >   the patches here adhere to that - there is far less testsuite > library >   support for host-side facilities (which we’d probably want to add > if >   it was

Re: (Re: [committed] openmp: Add nothing directive support)

2021-08-18 Thread Tobias Burnus
On 18.08.21 13:18, Jakub Jelinek wrote: gcc/testsuite/ChangeLog: * gfortran.dg/nothing-1.f90: New test. * gfortran.dg/nothing-2.f90: New test. While testing manually with -fopenmp, I did manage to place it for the testsuite run outside the 'gomp/' subdirectory. – Now fixed → attachme

Re: [PATCH] Move xx* builtins to vsx.md.

2021-08-18 Thread Bill Schmidt via Gcc-patches
Hi Mike, On 8/12/21 11:20 PM, Michael Meissner wrote: Move xx* builtins to vsx.md. I originally posted this patch in May. It needed a slight tune up as the souces have changed, so I'm reposting it now. I noticed that the xx built-in functions (xxspltiw, xxspltidp, xxsplti32dx, xxeval, xxblend

[PATCH] Allow explicitly specifying the thread model for runtime libs

2021-08-18 Thread John Ericson via Gcc-patches
From: John Ericson Previously, they always scraped the thread mode from `$CC -v', now, that is the default but one may pass `--with-threads=MODEL` to be explicit instead. One use-case is bootstraping with a shorter critical path. The traditionally route was to build an entire "static stage" GCC,

Re: [PATCH 1/2] analyzer: detect and analyze calls via function pointer (GSoC)

2021-08-18 Thread David Malcolm via Gcc-patches
On Wed, 2021-08-18 at 12:19 -0400, David Malcolm wrote: > On Tue, 2021-08-17 at 20:40 +0530, Ankur Saini wrote: > > Here is the final patch after fixing all the nits mentioned. > > > > Successfully bootstrapped and completed regress tests on x86_64- > > linux- > > gnu. > > Great, thanks! > > A

Re: [PATCH] Move xx* builtins to vsx.md.

2021-08-18 Thread David Edelsohn via Gcc-patches
On Fri, Aug 13, 2021 at 12:20 AM Michael Meissner wrote: > > Move xx* builtins to vsx.md. > > I originally posted this patch in May. It needed a slight tune up as the > souces have changed, so I'm reposting it now. > > I noticed that the xx built-in functions (xxspltiw, xxspltidp, xxsplti32dx, >

Re: [PATCH] PR fortran/100950 - ICE in output_constructor_regular_field, at varasm.c:5514

2021-08-18 Thread Harald Anlauf via Gcc-patches
Hi Tobias, > Gesendet: Mittwoch, 18. August 2021 um 12:22 Uhr > Von: "Tobias Burnus" > > Note, however, that gfc_simplify_len still won't handle neither > > deferred strings nor their substrings. > > > > I think there is nothing to simplify at compile time here. > > Obviously, nonsubstrings can

Re: [PATCH 1/6] rs6000: Support SSE4.1 "round" intrinsics

2021-08-18 Thread Segher Boessenkool
On Wed, Aug 18, 2021 at 02:01:33PM -0500, Bill Schmidt wrote: > On 8/9/21 3:23 PM, Paul A. Clarke via Gcc-patches wrote: > Reminder:  Please specify what testing you've done and on which > platforms, Yes please. > and where you want to apply the patches (i.e., do you want > backports?). Backpo

Re: [PATCH] testsuite, JIT, Darwin: Adjust asm tests for Mach-O.

2021-08-18 Thread David Malcolm via Gcc-patches
On Wed, 2021-08-18 at 20:45 +0100, Iain Sandoe wrote: > Hi, > > This provides adjusted assembler fragments that are suitable > for x86_64 Mach-O. > > ** as an aside, I wonder if it would be possible to configure some >    conditional on __ELF__, __MACH__, and CPU arch to pull in some >    object-

Re: [PATCH 1/6] rs6000: Support SSE4.1 "round" intrinsics

2021-08-18 Thread Segher Boessenkool
On Mon, Aug 09, 2021 at 03:23:50PM -0500, Paul A. Clarke wrote: > Suppress exceptions (when specified), by saving, manipulating, and > restoring the FPSCR. Similarly, save, set, and restore the floating-point > rounding mode when required. > > No attempt is made to optimize writing the FPSCR (by

Re: [PATCH] Move xx* builtins to vsx.md.

2021-08-18 Thread Segher Boessenkool
On Wed, Aug 18, 2021 at 04:42:42PM -0400, David Edelsohn wrote: > I wanted to give Segher a chance to comment on the structure. I think the current vector.md / altivec.md / vsx.md / rs6000.md division is artificial at best. Most of the basic (movement etc.) things are in rs6000.md (and all should

Re: [committed] Introduce selftest::locate_file (v5)

2021-08-18 Thread H.J. Lu via Gcc-patches
On Tue, Aug 17, 2021 at 12:01 AM Thomas Schwinge wrote: > > Hi! > > On 2016-12-14T21:31:05-0500, David Malcolm wrote: > > On Wed, 2016-12-14 at 15:02 +0100, Bernd Schmidt wrote: > >> On 12/09/2016 08:32 PM, David Malcolm wrote: > >> > Thanks. Unfortunately, applying the "locate_file" patch > >>

[r12-3003 Regression] FAIL: g++.dg/analyzer/pr96641.C -std=c++98 (test for excess errors) on Linux/x86_64

2021-08-18 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, 1b34248527472496ca3fe2a07183beac8cf69041 is the first bad commit commit 1b34248527472496ca3fe2a07183beac8cf69041 Author: Ankur Saini Date: Sun Aug 15 19:19:07 2021 +0530 analyzer: detect and analyze virtual function calls caused FAIL: g++.dg/analyzer/pr96641.C -std=c++1

[r12-3002 Regression] FAIL: gfortran.dg/analyzer/pr96949.f90 -O (test for excess errors) on Linux/x86_64

2021-08-18 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, aef703cf982072427e74034f4c460a11c5e04b8e is the first bad commit commit aef703cf982072427e74034f4c460a11c5e04b8e Author: Ankur Saini Date: Thu Jul 29 15:48:07 2021 +0530 analyzer: detect and analyze calls via function pointer caused FAIL: gcc.dg/analyzer/analyzer-verbosi

[PATCH, Fortran] Revert to non-multilib-specific ISO_Fortran_binding.h

2021-08-18 Thread Sandra Loosemore
This is a follow-up to commit fef67987cf502fe322e92ddce22eea7ac46b4d75: https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=fef67987cf502fe322e92ddce22eea7ac46b4d75 I realized last week that having multilib-specific versions of ISO_Fortran_binding.h (generated by running the compiler to ask what kin

Re: [PATCH] more warning code refactoring

2021-08-18 Thread David Edelsohn via Gcc-patches
Hi, Martin A few PowerPC-specific testcases started failing yesterday on AIX with a strange failure mode: the compiler runs out of memory. As you may expect from telling you this in an email reply to your patch, I have bisected the failure and landed on your commit. I can alternate between the p

[patch, libgfortran] Further fixes for GFC/CFI descriptor conversions

2021-08-18 Thread Sandra Loosemore
This patch addresses several bugs in converting from GFC to CFI descriptors and vice versa. It incorporates bits and pieces of an earlier patch from José https://gcc.gnu.org/pipermail/fortran/2021-June/056154.html that is now quite bit-rotten. The root of all the problems addressed here is t

Re: [PATCH v2] Fix incomplete computation in fill_always_executed_in_1

2021-08-18 Thread Xionghu Luo via Gcc-patches
On 2021/8/17 17:10, Xionghu Luo via Gcc-patches wrote: > > > On 2021/8/17 15:12, Richard Biener wrote: >> On Tue, 17 Aug 2021, Xionghu Luo wrote: >> >>> Hi, >>> >>> On 2021/8/16 19:46, Richard Biener wrote: On Mon, 16 Aug 2021, Xiong Hu Luo wrote: > It seems to me that ALWAYS_EXE

[PATCH v2] Don't move cold code out of loop by checking bb count

2021-08-18 Thread Xionghu Luo via Gcc-patches
On 2021/8/10 12:25, Ulrich Drepper wrote: > On Tue, Aug 10, 2021 at 4:03 AM Xionghu Luo via Gcc-patches > wrote: >> For this case, theorotically I think the master GCC will optimize it to: >> >>invariant; >>for (;;) >> if (unlikely_cond) >>for (;;) >> ; >> >> 'inv

Re: [PATCH] more warning code refactoring

2021-08-18 Thread Kewen.Lin via Gcc-patches
Hi David, on 2021/8/19 上午11:26, David Edelsohn via Gcc-patches wrote: > Hi, Martin > > A few PowerPC-specific testcases started failing yesterday on AIX with > a strange failure mode: the compiler runs out of memory. As you may > expect from telling you this in an email reply to your patch, I ha

Re: [PATCH] x86-64: Remove HAVE_LD_PIE_COPYRELOC

2021-08-18 Thread Fāng-ruì Sòng via Gcc-patches
PING^3 https://gcc.gnu.org/pipermail/gcc-patches/2021-May/570139.html On Fri, Jun 4, 2021 at 3:04 PM Fāng-ruì Sòng wrote: > > PING^2 https://gcc.gnu.org/pipermail/gcc-patches/2021-May/570139.html > > On Mon, May 24, 2021 at 9:43 AM Fāng-ruì Sòng wrote: > > > > Ping https://gcc.gnu.org/pipermail/

Re: [PATCH] c++, v3: Implement P0466R5 __cpp_lib_is_layout_compatible compiler helpers [PR101539]

2021-08-18 Thread Christophe Lyon via Gcc-patches
Hi Jakub, On Tue, Aug 17, 2021 at 5:35 PM Jason Merrill via Gcc-patches < gcc-patches@gcc.gnu.org> wrote: > On 8/17/21 10:55 AM, Jakub Jelinek wrote: > > On Tue, Aug 17, 2021 at 07:10:28AM -0700, Jason Merrill wrote: > >> Looks good, thanks. I think you didn't see that I also asked for some > a