Re: [PATCH] Use fold_build2 instead fold_binary for TRUTH_AND

2021-10-19 Thread Andrew Pinski via Gcc-patches
On Tue, Oct 19, 2021 at 7:30 PM Jiufu Guo via Gcc-patches wrote: > > In tree_simplify_using_condition_1, there is code which should be logic: > "op0 || op1"/"op0 && op1". When creating expression for TRUTH_OR_EXPR > and TRUTH_AND_EXPR, fold_build2 would be used instead fold_binary which > always

Re: [PATCH v2] tree-object-size: Make unknown a computation

2021-10-19 Thread Siddhesh Poyarekar
On 10/19/21 13:15, Jakub Jelinek wrote: Ok for trunk if it passes bootstrap/regtest. Pushed now, no new issues seen in bootstrap and regtest. Thanks, Siddhesh

Re: [PATCH] Objective-C: fix protocol list count type (pertinent to non-LP64)

2021-10-19 Thread Matt Jacobson via Gcc-patches
> On Sep 26, 2021, at 11:45 PM, Matt Jacobson wrote: > > Fix protocol list layout for non-LP64. clang and objc4 both give the `count` > field as `long`, not `intptr_t`. Those are the same on LP64, but not > everywhere. For non-LP64, this fixes binary compatibility with clang-built > class

[committed] Trivial fix to gil-1.c when analyzer is not enabled

2021-10-19 Thread Jeff Law via Gcc-patches
The test gcc.dg/plugin/gil-1.c should be reporting UNSUPPORTED when the analyzer is not enabled, but instead it's showing up as a FAIL.  This is because it's missing an appropriate dg-require-effective-target marker. Committed to the trunk, Jeff commit f36240f8c835d792f788b6724e272fc0a4a4f26f

Re: [PATCH] Use fold_build2 instead fold_binary for TRUTH_AND

2021-10-19 Thread guojiufu via Gcc-patches
On 2021-10-20 10:44, Andrew Pinski wrote: On Tue, Oct 19, 2021 at 7:30 PM Jiufu Guo via Gcc-patches wrote: In tree_simplify_using_condition_1, there is code which should be logic: "op0 || op1"/"op0 && op1". When creating expression for TRUTH_OR_EXPR and TRUTH_AND_EXPR, fold_build2 would be

[PATCH] X86: Add an option -muse-unaligned-vector-move

2021-10-19 Thread dianhong.xu--- via Gcc-patches
From: dianhong xu Add -muse-unaligned-vector-move option to emit unaligned vector move instaructions. gcc/ChangeLog: * config/i386/i386-options.c (ix86_target_string): Add -muse-unaligned-vector-move. * config/i386/i386.c (ix86_get_ssemov): Emit unaligned vector if use

[PATCH] i386: Fix wrong codegen for V8HF move without TARGET_AVX512F

2021-10-19 Thread Hongyu Wang via Gcc-patches
Since _Float16 type is enabled under sse2 target, returning V8HFmode vector without AVX512F target would generate wrong vmovdqa64 instruction. Adjust ix86_get_ssemov to avoid this. Bootstraped/regtested on x86_64-pc-linux-gnu{-m32,} and sde. OK for master? gcc/ChangeLog: PR target/102812

Re: [committed] libstdc++: Implement monadic operations for std::optional (P0798R8)

2021-10-19 Thread Tim Song via Gcc-patches
On Tue, Oct 19, 2021 at 9:05 AM Jonathan Wakely via Gcc-patches wrote: > > +constexpr bool > +test_copy_elision() > +{ > + return true; > +} > + > +static_assert( test_copy_elision() ); > + This isn't much of a test :)

Re: [PATCH v4] Fix ICE when mixing VLAs and statement expressions [PR91038]

2021-10-19 Thread Uecker, Martin
Am Montag, den 18.10.2021, 12:35 -0400 schrieb Jason Merrill: > On 10/17/21 09:52, Uecker, Martin wrote: > > > > Here is the 4th version of the patch. I tried to implement > > Jason's suggestion and this also fixes the problem. But > > I am not sure I understand the condition on > > the TREE_SIDE_

Re: [PATCH] x86: Adjust gcc.target/i386/pr22076.c

2021-10-19 Thread Uros Bizjak via Gcc-patches
On Tue, Oct 19, 2021 at 8:23 PM H.J. Lu wrote: > > commit 247c407c83f0015f4b92d5f71e45b63192f6757e > Author: Roger Sayle > Date: Mon Oct 18 12:15:40 2021 +0100 > > Try placing RTL folded constants in the constant pool. > > My recent attempts to come up with a testcase for my patch to ev

gfortran.dg/bind-c-contiguous-5.c: Big-endian fix [PR102815]

2021-10-19 Thread Tobias Burnus
Endian fix – but not well tested on big endian - hence, reports whether it now passes are highly welcome (thanks!). Committed as r12-4528-gd4044db034b40c275b5f287d5854a102d22e07c0 Tobias - Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Ges

Re: [PATCH 4/4] Improve maybe_remove_writeonly_store to do a simple DCE for defining statement

2021-10-19 Thread Richard Biener via Gcc-patches
On Wed, Oct 20, 2021 at 1:14 AM Jeff Law via Gcc-patches wrote: > > > > On 10/18/2021 10:54 PM, apinski--- via Gcc-patches wrote: > > From: Andrew Pinski > > > > Instead of putting a full blow DCE after execute_fixup_cfg, it makes sense > > to try to remove the defining statement for the store th

<    1   2