On Tue, Jul 21, 2020 at 11:14 AM Jojo wrote:
>
> gcc/ChangeLog:
>
> * genemit.c (main): Print 'split line'.
> * Makefile.in (insn-emit.c): Define split count and file
>
Thanks for working one this, following comments are based on the
assumption that the approach is feasible after
- Verifed on RISC-V and x86.
gcc/testsuite/ChangeLog:
* gcc.dg/analyzer/signal-1.c: Add dg-require-effective-target
signal.
* gcc.dg/analyzer/signal-2.c: Ditto.
* gcc.dg/analyzer/signal-3.c: Ditto.
* gcc.dg/analyzer/signal-4a.c: Ditto.
* gcc.dg/ana
* Jonathan Wakely via Libstdc:
> By replacing the use of strtod we could avoid allocation, avoid changing
> locale, and use optimised code paths specific to each std::chars_format
> case. We would also get more portable behaviour, rather than depending
> on the presence of uselocale, and on any bu
Hi,
This patch is to add the cost modeling for vector with length,
it mainly follows what we generate for vector with length in
functions vect_set_loop_controls_directly and vect_gen_len
at the worst case.
For Power, the length is expected to be in bits 0-7 (high bits),
we have to model the cost
On Mon, Jul 20, 2020 at 5:06 AM Richard Biener
wrote:
>
> On Sat, Jul 18, 2020 at 7:57 AM Sunil Pandey wrote:
> >
> > On Fri, Jul 17, 2020 at 1:22 AM Richard Biener
> > wrote:
> > >
> > > On Fri, Jul 17, 2020 at 7:15 AM Sunil Pandey wrote:
> > > >
> > > > Any comment on revised patch? At least,
If the value does not contain any path component separator (e.g. a
slash), the linker will be searched for using COMPILER_PATH followed by
PATH. Otherwise, it is either an absolute path or a path relative to the
current working directory.
--ld-path= complements and overrides -fuse-ld={bfd,gold,lld
On 2020/7/20 23:31, Segher Boessenkool wrote:
On Mon, Jul 13, 2020 at 02:30:28PM +0800, luoxhu wrote:
For extracting high part element from DImode register like:
{%1:SF=unspec[r122:DI>>0x20#0] 86;clobber scratch;}
split it before reload with "and mask" to avoid generating shift right
32 bit th
gcc/ChangeLog:
* genemit.c (main): Print 'split line'.
* Makefile.in (insn-emit.c): Define split count and file
---
gcc/Makefile.in | 10 ++
gcc/genemit.c | 86 +++--
2 files changed, 58 insertions(+), 38 deletions(-)
diff --git
Hi Segher,
on 2020/7/21 上午12:58, Segher Boessenkool wrote:
> Hi!
>
> On Fri, Jul 10, 2020 at 06:07:16PM +0800, Kewen.Lin wrote:
>> +/* { dg-do compile { target { powerpc*-*-* } && { lp64 &&
>> powerpc_p9vector_ok } } } */
>
> Everything in gcc.targer/powerpc/ requires powerpc*-*-* automatically
Regular ELF label definitions for this test-case, matched by the
regexps, e.g.:
/* { dg-final { scan-assembler-times {(?n)^_*bar[.$_]constprop[.$_]0:} 1 } } */
typically look like this:
bar_constprop.0:
For MMIX, they look like this:
bar_constprop::0IS @
I think it's better to just skip
I missed updating the line-number when adding that dg-skip-if.
Committed as obvious.
* gcc.dg/cdce3.c: Update matched line-number.
diff --git a/gcc/testsuite/gcc.dg/cdce3.c b/gcc/testsuite/gcc.dg/cdce3.c
index 71aea9b..601ddf0 100644
--- a/gcc/testsuite/gcc.dg/cdce3.c
+++ b/gcc/testsuite/
P1975R0 tweaks the static_cast wording: it says that "An expression e can be
explicitly converted to a type T if [...] T is an aggregate type having a first
element x and there is an implicit conversion sequence from e to the type of
x." This already works for classes, e.g.:
struct Aggr { int x
MMIX has two stacks; the regular one using register $254 as a
convention and the register-stack, pushed and popped by call
instructions (usually). The decision to only report the stack usage
of the regular stack (and not of the register stack) may be updated,
perhaps the sum is better. This initi
On 7/20/20 2:15 AM, Richard Biener wrote:
I think at least parts of tree-prof.exp exercises sample-based profiling
which might require more iterations. For example cold_partition_label.c
was changed by
commit f63ba78ce6d50bf627dd18018179eb03bf89716f
Author: Andi Kleen
Date: Thu Jul 14 02:14
This adds the missing std::from_chars overloads for floating-point
types, as required for C++17 conformance.
The implementation is a hack and not intended to be used in the long
term. Rather than parsing the string directly, this determines the
initial portion of the string that matches the patter
This patch adds few helper functions aims to improve readability of
fetching addresses, sizes, and values. It also proposes a syntax for
querying these information through the callback functions, similar to
that of LLVM implementation. The syntax format is _, or __. '_' is the
delimiter between fie
P0593R6 is mostly about a new object model whereby malloc and the like are
treated as implicitly starting the lifetime of whatever trivial types are
necessary to give the program well-defined semantics; that seems only
relevant to TBAA, and is not implemented here.
The paper also specifies that a
On Fri, 17 Jul 2020 10:19:41 +0200
Eric Botcazou wrote:
> > Oh! Sent out v3 with tweaked description as
> > https://gcc.gnu.org/pipermail/gcc-patches/2020-July/550168.html
>
> Thanks.
>
> > I don't have a push access to gcc tree. Should I request one via
> > https://sourceware.org/cgi
The last new thing allowed by P1907R1: subobject addresses as template
arguments. The ABI group has discussed mangling for this; there has been
some talk of a compressed subobject mangling, but it hasn't been finalized,
so for now I'm using normal expression mangling. In order for two array
subob
In CWG discussion, it was suggested that deduction from a string literal
should be to reference-to-const, so that we deduce 'char' rather than 'const
char' for T.
Tested x86_64-pc-linux-gnu, applying to trunk.
gcc/cp/ChangeLog:
* pt.c (collect_ctor_idx_types): Add 'const' when deducing f
On 7/20/20 9:12 PM, Jakub Jelinek wrote:
I don't like this global variable.
Can you please instead stick it into struct nesting_info and make sure it is
cleared where it is allocated?
Done. The existing code uses
struct nesting_info *info = XCNEW (struct nesting_info);
in create_nesting_tree
In the below testcase, duplicate_decls wasn't merging the tsubsted
friend declaration for 'void add(auto)' with its definition, because
reduce_template_parm_level (during tsubst_friend_function) lost the
DECL_VIRTUAL_P flag on the invented 'auto' template parameter, which
made template_heads_equiva
On Mon, Jul 20, 2020 at 08:35:56PM +0200, Tobias Burnus wrote:
> gcc/ChangeLog:
>
> PR fortran/93553
> * tree-nested.c (omp_new_clauses): New global var.
> (convert_nonlocal_reference_op): Add init_tmp_var/init_tmp_var
> vars to it.
> (convert_nonlocal_omp_clauses, co
Similar to the recent changes to basic_istream::ignore, this change
ensures that _M_gcount doesn't overflow when extracting characters and
inserting them into another streambuf.
The solution used here is to use unsigned long long for the count. We
assume that the number of characters extracted won
Sorry for the slow reply.
"yangyang (ET)" writes:
> Hi,
>
> This is a simple fix for PR96195.
>
> For the test case, GCC generates the following gimple statement in
> pass_vect:
>
> vect__21.16_58 = zp.simdclone.2 (vect_vec_iv_.15_56);
>
> The mode of vect__21.16_58 is
I have committed this trivial change below in r11-2229.
Martin
diff --git a/gcc/c-family/c.opt b/gcc/c-family/c.opt
index a09f15d..2b1aca1 100644 (file)
--- a/gcc/c-family/c.opt
+++ b/gcc/c-family/c.opt
@@ -1248,7 +1248,6 @@ C ObjC C++ LTO ObjC++ Var(warn_vla_limit) Warning
Joined Host_Wide_Int
I have committed this change in r11-2231 after Jeff approved it
off list last Thursday.
On 6/30/20 6:23 PM, Martin Sebor wrote:
An enhancement to GCC 10 to improve the expansion of strncmp
calls with strings with embedded nuls introduced a regression
in similar calls to memcmp. A review of the
Currently, __atomic_thread_fence(seq_cst) on x86 and x86-64 generates
mfence instruction. A dummy atomic instruction (a lock-prefixed instruction
or xchg with a memory operand) would provide the same sequential consistency
guarantees while being more efficient on most current CPUs. The mfence
instr
This is about a PARAM_DECL of a procedure whose
internal/nested procedure uses this inside an
omp parallel. This leads to the code:
D.3940 = x;
(*D.3940)[D.3924] = …;
And the temporary variable "D.3940" introduced for
the nesting was not recorded as DECL for OpenMP,
leading to the ICE in scan_om
Dimitar Dimitrov writes:
> Fix test cases assumptions that target has alignment constraints.
>
> gcc/testsuite/ChangeLog:
>
> * gcc.dg/attr-copy-4.c: Unpacked may still have alignment of 1
> on targets with default_packed.
> * gcc.dg/c11-align-9.c: Remove AVR target filter and re
Dimitar Dimitrov writes:
> The actual warning message depends on the default alignment of the
> target. With this update the test correctly passes on AVR and PRU
> targets.
>
> gcc/testsuite/ChangeLog:
>
> * gcc.dg/pr53037-1.c: Relax warning pattern.
>
> Signed-off-by: Dimitar Dimitrov
> --
Dimitar Dimitrov writes:
> Targets which pack structures by default get warnings for packed structure
> attributes. This is expected, so add markers in the test cases.
>
> gcc/testsuite/ChangeLog:
>
> * c-c++-common/Waddress-of-packed-member-2.c: Add dg-warning for
> ignored attribute
Dimitar Dimitrov writes:
> Targets which pack structures by default will not get warnings about
> unaligned access to structure members.
>
> gcc/testsuite/ChangeLog:
>
> * c-c++-common/Waddress-of-packed-member-1.c: Filter dg-warning
> for targets who pack by default.
> * c-c++-c
Hi!
On Fri, Jul 10, 2020 at 06:07:16PM +0800, Kewen.Lin wrote:
> +/* { dg-do compile { target { powerpc*-*-* } && { lp64 &&
> powerpc_p9vector_ok } } } */
Everything in gcc.targer/powerpc/ requires powerpc*-*-* automatically
(is never run on other targets).
> +/* { dg-final { scan-assembler-tim
Hi,
I'm sending a few minor testsuite updates to add markers for targets using
packed structures by default. From those targets, I tested AVR and PRU. I don't
have setup to test cris and m32c.
I also tested x86_64 to ensure there are neither dropped nor newly failing
tests.
Regards,
Dimitar
Fix test cases assumptions that target has alignment constraints.
gcc/testsuite/ChangeLog:
* gcc.dg/attr-copy-4.c: Unpacked may still have alignment of 1
on targets with default_packed.
* gcc.dg/c11-align-9.c: Remove AVR target filter and replace
with default_packe
Targets which pack structures by default get warnings for packed structure
attributes. This is expected, so add markers in the test cases.
gcc/testsuite/ChangeLog:
* c-c++-common/Waddress-of-packed-member-2.c: Add dg-warning for
ignored attribute if target is default_packed.
The actual warning message depends on the default alignment of the
target. With this update the test correctly passes on AVR and PRU
targets.
gcc/testsuite/ChangeLog:
* gcc.dg/pr53037-1.c: Relax warning pattern.
Signed-off-by: Dimitar Dimitrov
---
gcc/testsuite/gcc.dg/pr53037-1.c | 4 +
Targets which pack structures by default will not get warnings about
unaligned access to structure members.
gcc/testsuite/ChangeLog:
* c-c++-common/Waddress-of-packed-member-1.c: Filter dg-warning
for targets who pack by default.
* c-c++-common/Waddress-of-packed-member-2.
On Jul 18, 2020, at 8:19 PM, Hans-Peter Nilsson wrote:
>
> Long-standing FAIL remedied; committed. Maybe better to list
> the targets that *do* support arbitrary frame access?
Yes, it would be better if test cases that fall way to low in portability are
listed instead against what platforms th
Hi,
Zhongyunde writes:
> Hi，
>
> In most target, it is limited to issue two insns with change the same
> register. So a register is not realy unused if there is another insn, which
> set the register in the save VLIW.
>
> For example, The insn 73 start with insn:TI, so it will be issued togethe
On Mon, Jul 13, 2020 at 02:30:28PM +0800, luoxhu wrote:
> For extracting high part element from DImode register like:
>
> {%1:SF=unspec[r122:DI>>0x20#0] 86;clobber scratch;}
>
> split it before reload with "and mask" to avoid generating shift right
> 32 bit then shift left 32 bit. This pattern a
Hi David, Jakub, Tobias, gfortran/OMP developers!
This is about how an OpenACC (but also OpenMP 'target', I suppose) data
clause for Fortran 'allocatable' with array descriptor appears in the
'original' dump.
On 2020-07-18T16:02:12+, David Edelsohn wrote:
> https://gcc.gnu.org/g:60c1baebbaa
On Mon, 20 Jul 2020, Jonathan Wakely wrote:
> On 20/07/20 08:53 -0400, Patrick Palka via Libstdc++ wrote:
> > On Mon, 20 Jul 2020, Jonathan Wakely wrote:
> >
> > > On 19/07/20 23:37 -0400, Patrick Palka via Libstdc++ wrote:
> > > > On Fri, 17 Jul 2020, Patrick Palka wrote:
> > > >
> > > > > On Fr
On Mon, 2020-07-20 at 12:10 +0100, Alex Coplan wrote:
> Hello,
>
> This patch fixes a bug in jit.exp which causes the DejaGnu output of
> the
> libgccjit testsuite to be nondeterministically truncated. This bug
> was
> copied from DejaGnu's own implementation of the host_execute
> function.
> See
On 7/18/20 5:11 PM, Jakub Jelinek wrote:
On Sat, Jul 18, 2020 at 05:04:56PM -0400, David Edelsohn via Gcc-patches wrote:
H-P,
After your patch to the testsuite, the cpp/pragma-eof.c testcase is
failing on all targets. Would you please investigate and fix?
That is because the dg-error directi
* Richard Biener via Gcc-patches:
> Given you know the constant bit position of the possibly nonzero
> bit you can elide the conversion to unsigned for all but the case
> of a possibly negative input (IIRC GCC doesn't yet take advantage
> of negative right shift undefinedness - but maybe sanitizer
On 20/07/20 08:53 -0400, Patrick Palka via Libstdc++ wrote:
On Mon, 20 Jul 2020, Jonathan Wakely wrote:
On 19/07/20 23:37 -0400, Patrick Palka via Libstdc++ wrote:
> On Fri, 17 Jul 2020, Patrick Palka wrote:
>
> > On Fri, 17 Jul 2020, Patrick Palka wrote:
> >
> > > On Wed, 15 Jul 2020, Patrick
Hu Jiangping writes:
> Hi,
>
> This patch deal with the -falign-X=0 options. According to man pages,
> if zero is specified, a machine-dependent default value should be used.
> But in fact, zero was used in internal process, it is inconsistent.
>
> Tested on aarch64-linux cross compiler, Is that O
This produces a more optimal RPO order for iteration processing
by making sure that SCC members are processed before blocks reachable
from SCC exits. This avoids iterating blocks unrelated to the current
iteration for RPO VN.
Overall reduction in the number of visited blocks isn't spectacular
for
This removes a write-only array in
rev_post_order_and_mark_dfs_back_seme.
Bootstrapped / tested on x86_64-unknown-linux-gnu.
2020-07-20 Richard Biener
* cfganal.c (rev_post_order_and_mark_dfs_back_seme): Remove
write-only post array.
---
gcc/cfganal.c | 13 +
1 fi
On Mon, Jul 20, 2020 at 3:06 PM Roger Sayle wrote:
>
>
> This patch complements one from June 12th which is still awaiting
> review: https://gcc.gnu.org/pipermail/gcc-patches/2020-June/547937.html
>
> This patch optimizes popcount and parity of an argument known to have
> at most a single bit set,
On 08/07/2020 09:04, Andre Simoes Dias Vieira wrote:
On 07/07/2020 13:43, Christophe Lyon wrote:
Hi,
On Mon, 6 Jul 2020 at 16:31, Andre Vieira (lists)
wrote:
On 30/06/2020 14:50, Andre Vieira (lists) wrote:
On 29/06/2020 11:15, Christophe Lyon wrote:
On Mon, 29 Jun 2020 at 10:56, Andre
On Wed, Jul 15, 2020 at 5:40 PM Dmitrij Pochepko
wrote:
>
> Hi,
>
> here is an enhancement to gcc, which allows load/store groups with size being
> non-power-of-2 to be vectorized.
> Current implementation is using interleaving permutations to transform
> load/store groups. That is where power-o
On 01/07/2020 3:28 pm, Tom de Vries wrote:
So, I think gcc needs a copy of (some of) the
gcc/testsuite/gcc.dg/ia64-sync-*.c tests for effective target
sync_char_short.
However, since this patch only adds partial support, we cannot enable
sync_char_short for nvptx yet. So, if you stick to partia
This patch complements one from June 12th which is still awaiting
review: https://gcc.gnu.org/pipermail/gcc-patches/2020-June/547937.html
This patch optimizes popcount and parity of an argument known to have
at most a single bit set, to be that single bit. Hence, popcount(x&8)
is simplified to (
On Mon, 20 Jul 2020, Jonathan Wakely wrote:
> On 19/07/20 23:37 -0400, Patrick Palka via Libstdc++ wrote:
> > On Fri, 17 Jul 2020, Patrick Palka wrote:
> >
> > > On Fri, 17 Jul 2020, Patrick Palka wrote:
> > >
> > > > On Wed, 15 Jul 2020, Patrick Palka wrote:
> > > >
> > > > > On Tue, 14 Jul 202
On 19/07/20 23:37 -0400, Patrick Palka via Libstdc++ wrote:
On Fri, 17 Jul 2020, Patrick Palka wrote:
On Fri, 17 Jul 2020, Patrick Palka wrote:
> On Wed, 15 Jul 2020, Patrick Palka wrote:
>
> > On Tue, 14 Jul 2020, Patrick Palka wrote:
> >
> > > This implements the floating-point std::to_chars
I found the linemap logic dealing with running out of column numbers
confusing. There's no need for completely separate code blocks there,
as we can rely on the masking operations working all the way down to
zero bits. The two binary searches for linemap lookups could do with
modernization of
I found the linemap logic dealing with running out of column numbers
confusing. There's no need for completely separate code blocks there,
as we can rely on the masking operations working all the way down to
zero bits. The two binary searches for linemap lookups could do with
modernization of
On Mon, Jul 20, 2020 at 1:04 PM Andrew Stubbs wrote:
>
> On 20/07/2020 11:01, Richard Biener wrote:
> > On Mon, Jul 20, 2020 at 10:40 AM Andrew Stubbs
> > wrote:
> >>
> >> On 20/07/2020 08:35, Richard Biener wrote:
> The way simple_object is supposed to work is to clone (or merge) the ELF
>
On Sat, Jul 18, 2020 at 7:57 AM Sunil Pandey wrote:
>
> On Fri, Jul 17, 2020 at 1:22 AM Richard Biener
> wrote:
> >
> > On Fri, Jul 17, 2020 at 7:15 AM Sunil Pandey wrote:
> > >
> > > Any comment on revised patch? At least, in finish_decl, decl global
> > > attributes are populated.
> >
> > +s
Hello,
This patch fixes a bug in jit.exp which causes the DejaGnu output of the
libgccjit testsuite to be nondeterministically truncated. This bug was
copied from DejaGnu's own implementation of the host_execute function.
See the upstream bug report [0] where the maintainers point out that the
reg
On 20/07/2020 11:01, Richard Biener wrote:
On Mon, Jul 20, 2020 at 10:40 AM Andrew Stubbs wrote:
On 20/07/2020 08:35, Richard Biener wrote:
The way simple_object is supposed to work is to clone (or merge) the ELF
headers from an existing binary. Unfortunately, the way mkoffload is
currently c
Thomas Schwinge writes:
Hi Thomas,
>> Can I include the patch in OG10?
>
> Unless Julian/Kwok speak up soon: OK, thanks.
This has been delayed a bit by my vacation, but I have now committed
the patch.
> May want to remove "libgomp" from the first line of the commit log --
> this commit doesn't
On Mon, Jul 20, 2020 at 10:40 AM Andrew Stubbs wrote:
>
> On 20/07/2020 08:35, Richard Biener wrote:
> >> The way simple_object is supposed to work is to clone (or merge) the ELF
> >> headers from an existing binary. Unfortunately, the way mkoffload is
> >> currently coded we don't have any to clo
On 20/07/2020 08:35, Richard Biener wrote:
The way simple_object is supposed to work is to clone (or merge) the ELF
headers from an existing binary. Unfortunately, the way mkoffload is
currently coded we don't have any to clone from until too late. We could
separate the assemble and link steps, b
Correct PR number in ChangeLog
it's pr96243.
On Mon, Jul 20, 2020 at 1:46 PM Hongtao Liu wrote:
>
> Hi:
> For rtx like (eq:HI (V8SI 90) (V8SI 91)), cse will take it as a
> boolean value and try to do some optimization. But it is not true for
> vector compare, also other places in rtl passes hol
Hu Jiangping writes:
> Hi,
>
> Different from Patch v1 which removed the page, I add a big banner to
> say that the page is no longer up-to-date, just as Richard and Gerald
> recommended. OK for push?
Thanks, pushed to wwwdocs. Sorry for the slow response.
Richard
On Sat, 18 Jul 2020, Jakub Jelinek wrote:
> Hi!
>
> When working on __builtin_bit_cast that needs to handle bitfields too,
> I've made the following change to handle at least some bitfields in
> fold_const_aggregate_ref_1 (those that have integral representative).
> It already handles some, but o
On Mon, Jul 20, 2020 at 6:48 AM Sandra Loosemore
wrote:
>
> I was looking at some timeout failures in nios2-linux-gnu test results
> and found several tree-prof.exp tests were doing what appears to be an
> excessive number of iterations (350 million?). Even though this is
> hardware and not a sim
On Sat, 18 Jul 2020, Jakub Jelinek wrote:
> Hi!
>
> When working on __builtin_bit_cast that needs to handle bitfields too,
> I've made the following change to handle at least some bitfields in
> native_encode_initializer (those that have integral representative).
>
> Bootstrapped/regtested on {x
On Fri, Jul 17, 2020 at 1:30 PM Andrew Stubbs wrote:
>
> On 17/07/2020 07:20, Thomas Schwinge wrote:
> >> --- a/gcc/config/gcn/mkoffload.c
> >> +++ b/gcc/config/gcn/mkoffload.c
> >> @@ -33,31 +33,53 @@
> >> #include
> >> #include "collect-utils.h"
> >> #include "gomp-constants.h"
> >> +#inc
Please find attached a fix for PR53298.
This appears to be a very simple fix, however, since it involves
structures I'm unfamiliar with I think it needs checking.
When the gfc_ref structure is created for a (1:) substring it has an
expression for start and a NULL for the end. For (:5) the st
74 matches
Mail list logo
