On Mon, 22 Feb 2021, Uros Bizjak wrote:
> The intention of HONOR_REG_ALLOC_ORDER is to ensure that IRA allocates
> registers in the order given by REG_ALLOC_ORDER. However in
> ira_better_spill_reload_regno_p, there is still a place where the
> calculation depends on the presence of
Hi Kito,
Kito Cheng writes:
> FAIL: gcc.c-torture/execute/string-opt-5.c
> FAIL: gcc.target/riscv/shift-and-1.c scan-assembler-not andi
> FAIL: gcc.target/riscv/shift-and-2.c scan-assembler-not andi
string-opt-5.c is one of the newlib issues I mentioned (handcoded
assembler for strcmp which
Seems like only 3 fail are related to big-endian, you don't need to
worry about other fails.
FAIL: gcc.c-torture/execute/string-opt-5.c
FAIL: gcc.target/riscv/shift-and-1.c scan-assembler-not andi
FAIL: gcc.target/riscv/shift-and-2.c scan-assembler-not andi
On Tue, Feb 23, 2021 at 10:38 AM Kito
This is a backport of adding cost tables for A64FX.
Bootstrapped and tested on aarch64-none-linux-gnu.
2021-02-23 Qian Jianhua
gcc/ChangeLog:
* config/aarch64/aarch64-cost-tables.h (a64fx_extra_costs): New.
* config/aarch64/aarch64.c (a64fx_addrcost_table): New.
From: Aaron Sawdey
This patch implements a RTL pass that looks for pc-relative loads of the
address of an external variable using the PCREL_GOT relocation and a
single load or store that uses that external address.
Produced by a cast of thousands:
* Michael Meissner
* Peter Bergner
* Bill
On 2/13/21 10:21 PM, Patrick Palka wrote:
On Fri, 12 Feb 2021, Patrick Palka wrote:
Here, the problem ultimately seems to be that tsubst_copy_and_build,
when called with empty args as we do during non-dependent expression
folding, doesn't touch BASELINKs at all: it delegates to tsubst_copy
On 2/15/21 7:45 AM, Anthony Sharp wrote:
PARENT_BINFO, has private access to DECL. Examine certain sepcial cases
"special"
I did try the name
lookup as Jason suggested but, as I say, in the case of overloaded
functions it returns multiple values, so it didn't help in determining
what
For MIPS N64 and N32:
add GNATRTL_128BIT_PAIRS to LIBGNAT_TARGET_PAIRS
add GNATRTL_128BIT_OBJS to EXTRA_GNATRTL_NONTASKING_OBJS
gcc/ada/ChangeLog:
PR ada/98996
* Makefile.rtl: add 128Bit operation file to MIPS
N64 and N32 (LIBGNAT_TARGET_PAIRS,
It is found by ada s-pack96.adb ftbfs, due to 96bit load: 96 = 64 + 32.
While the 32bit pair of l r is mark as SUBREG, so they are
not in SImode, make it fail to find suitable insn.
gcc/ChangeLog:
* config/mips/mips.c (mips_expand_ext_as_unaligned_load):
If TARGET_64BIT and dest is
Hi Marcus:
Thanks for the quick update, I am testing your V2 patch now, the result seems
really great now, some of fail case seems like not cause by
big-endian patch, I am reviewing and comparing the fail case with the
little-endian build.
> Should I make a PR against riscv-newlib on GitHub, or
On 2/12/21 6:36 PM, Patrick Palka wrote:
This makes instantiation_dependent_expression_p avoid checking
potential_constant_expression when processing_template_decl isn't set
(and hence when value_dependent_expression_p is definitely false).
OK.
gcc/cp/ChangeLog:
* pt.c
On 2/22/21 3:59 PM, Iain Sandoe wrote:
Hi,
PR96251 (and dups) is a rejects-valid bug against coroutines (but I’m
not sure that the problem is really there).
* coroutines cannot be constexpr.
* Part of the way that the coroutine implementation works is a
consequence of the "lazy discovery of
On 2/22/21 8:00 PM, Martin Sebor wrote:
On 2/22/21 4:08 PM, Jason Merrill wrote:
On 2/13/21 7:31 PM, Martin Sebor wrote:
The test case in PR 99074 invokes dynamic_cast with the this pointer
in a non-static member function called on a null pointer. The call
is, of course, undefined and other
Hi Harald, Looks Good to Me.
OK
Thanks
On 2/22/21 1:14 PM, Harald Anlauf via Fortran wrote:
Dear all,
when simplifying the RESHAPE intrinsic we lost the string length when
the resulting array had size zero. The attached patch sets the resulting
length from the source.
Regtested on
> From: Martin Sebor
> Date: Tue, 23 Feb 2021 02:07:59 +0100
> On 2/22/21 5:48 PM, Hans-Peter Nilsson wrote:
> > Looking at commit de05c19d5fd6, that adjustment to these
> > tests apparently assumed that the testsuite is run (only) on
> > targets where structure memory layout has padding as per
On 2/22/21 5:48 PM, Hans-Peter Nilsson wrote:
Looking at commit de05c19d5fd6, that adjustment to these
tests apparently assumed that the testsuite is run (only) on
targets where structure memory layout has padding as per
"natural alignment". For cris-elf, a target with no padding
in structure
On 2/22/21 4:08 PM, Jason Merrill wrote:
On 2/13/21 7:31 PM, Martin Sebor wrote:
The test case in PR 99074 invokes dynamic_cast with the this pointer
in a non-static member function called on a null pointer. The call
is, of course, undefined and other different circumstances would be
diagnosed
Looking at commit de05c19d5fd6, that adjustment to these
tests apparently assumed that the testsuite is run (only) on
targets where structure memory layout has padding as per
"natural alignment". For cris-elf, a target with no padding
in structure memory layout, these tests have been failing
Ping 4:
https://gcc.gnu.org/pipermail/gcc-patches/2021-January/564059.html
On 2/14/21 5:43 PM, Martin Sebor wrote:
Ping 3:
https://gcc.gnu.org/pipermail/gcc-patches/2021-January/564059.html
This is a fix for two P2 bugs (false positives).
On 2/6/21 10:13 AM, Martin Sebor wrote:
Ping 2:
On Mon, 22 Feb 2021 at 16:30, Segher Boessenkool
wrote:
>
> Hi!
>
> First off, thanbk you for the patch!
You're welcome!
> On Mon, Feb 15, 2021 at 11:22:52PM +, Neven Sajko via Gcc-patches wrote:
> > There is a long-standing, but undocumented GCC inline assembly feature
> > that's part of
PR analyzer/99196 describes a false positive from -fanalyzer due to
the analyzer not "knowing" that calls to GNU libc's error(3) with a
nonzero status terminate the process and thus don't return.
This patch fixes the false positive by special-casing "error" and
"error_at_line".
Successfully
On Mon, Feb 22, 2021 at 03:06:58PM -0800, H.J. Lu via Gcc-patches wrote:
> Since retain attribute requires SHF_GNU_RETAIN, run retain tests only
> if SHF_GNU_RETAIN is supported.
>
> PR testsuite/99173
> * c-c++-common/attr-retain-5.c: Require R_flag_in_section.
> *
Changes in this version from Version 7:
gcc/c-family/c-cppbuiltin.c
Changed modename to float_h_prefix
Removed (mode == TYPE_MODE...) code left over from earlier approach.
libgcc/libgcc2.c
Removed conditional use of XF constants in TF case.
Also left over
On 2/13/21 7:31 PM, Martin Sebor wrote:
The test case in PR 99074 invokes dynamic_cast with the this pointer
in a non-static member function called on a null pointer. The call
is, of course, undefined and other different circumstances would be
diagnosed by -Wnonnull. Unfortunately, in the
Since retain attribute requires SHF_GNU_RETAIN, run retain tests only
if SHF_GNU_RETAIN is supported.
PR testsuite/99173
* c-c++-common/attr-retain-5.c: Require R_flag_in_section.
* c-c++-common/attr-retain-6.c: Likewise.
* c-c++-common/attr-retain-7.c: Likewise.
I decided to use float_h_prefix instead of modename or mode prefix
as Joseph has better insight as to naming expectations when
others are working in gcc/c-family/c-cppbuiltin.c.
All requested changes made and tested. Patch coming shortly.
- patrick
On 2/19/2021 5:48 PM, Joseph Myers wrote:
On 2/18/21 7:15 AM, Martin Liška wrote:
We crash when target attribute get_function_versions_dispatcher is called
for a function that is not registered in call graph. This fixes that
by emitting a new error.
Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
Ready to be
On Mon, 22 Feb 2021, 22:13 Patrick Palka via Libstdc++, <
libstd...@gcc.gnu.org> wrote:
> The logic in std::to_chars for extracting the high- and low-order parts
> of a IBM long double value does the right thing on powerpc64le, but not
> on powerpc64be. This patch makes the extraction
This makes the hexadecimal section of the long double std::to_chars
testcase more robust by avoiding false-negative FAILs due to printf
using a different leading hex digit than us, and by additionally
verifying the correctness of the hexadecimal form via round-tripping
through std::from_chars.
The logic in std::to_chars for extracting the high- and low-order parts
of a IBM long double value does the right thing on powerpc64le, but not
on powerpc64be. This patch makes the extraction endian-agnostic, which
fixes the execution FAIL of to_chars/long_double.cc on powerpc64be.
Tested on
From: Thomas Rodgers
This is a substantial rewrite of the atomic wait/notify (and timed wait
counterparts) implementation.
The previous __platform_wait looped on EINTR however this behavior is
not required by the standard. A new _GLIBCXX_HAVE_PLATFORM_WAIT macro
now controls whether wait/notify
> -Original Message-
> From: Gcc-patches On Behalf Of
> Kyrylo Tkachov via Gcc-patches
> Sent: 16 February 2021 15:20
> To: gcc-patches@gcc.gnu.org
> Subject: [PATCH] aarch64: Add internal tune flag to minimise VL-based scalar
> ops
>
> Hi all,
>
> This patch introduces an internal
On Mon, Feb 22, 2021 at 03:52:52PM -0500, Michael Meissner wrote:
> On Sat, Feb 20, 2021 at 06:33:12PM -0600, Segher Boessenkool wrote:
> > So if built with a glibc version before 2.32 (less than a year old) it
> > will give the wrong answer. This needs improving, or it will be another
> > eight
Dear all,
when simplifying the RESHAPE intrinsic we lost the string length when
the resulting array had size zero. The attached patch sets the resulting
length from the source.
Regtested on x86_64-pc-linux-gnu. OK for mainline?
Thanks,
Harald
PR fortran/99206 - ICE in add_init_expr_to_sym,
Hi,
PR96251 (and dups) is a rejects-valid bug against coroutines (but I’m
not sure that the problem is really there).
* coroutines cannot be constexpr.
* Part of the way that the coroutine implementation works is a
consequence of the "lazy discovery of coroutine-ness”; whenever
we first
On Sat, Feb 20, 2021 at 06:33:12PM -0600, Segher Boessenkool wrote:
> Hi!
>
> On Tue, Feb 09, 2021 at 02:35:05AM -0500, Michael Meissner wrote:
> > This patch implements conversions between _Float128 and the 3 Decimal
> > floating
> > types. It does this by extendending the dfp-bit conversions
See gcc/config/newlib-stdint.h, where targets that have
LONG_TYPE_SIZE == 32, get __INT32_TYPE__ defined to "long
int".
All these tests have "typedef __INT32_TYPE__ int32_t;".
Thus the tests fail for 32-bit newlib targets due to related
warning messages being matched to "aka int" where the
- Original Message -
> Hi!
>
> In GCC 10, parallel_backend.h just included parallel_backend_{serial,tbb}.h
> and
> did nothing beyond that, and parallel_backend_tbb.h provided directly
> namespace __pstl { namespace __par_backend { ... } }
> and defined everything in there, while
On Mon, Feb 22, 2021 at 02:49:44PM +0100, Jakub Jelinek wrote:
> So, I think for the team != gomp_thread ()->ts.team
> && !do_wake
> && gomp_team_barrier_waiting_for_tasks (>barrier)
> && team->task_detach_count == 0
> case, we need to wake up 1 thread anyway and arrange for it to do:
>
Also, tweak the scan-assembler regexps to include a tab,
lest they may spuriously match file-paths in the emitted
assembly code, should some be added at some point. And, add
"mul", "move" and (non-addi-)"add" to insns that shouldn't
appear.
Committed.
gcc/testsuite:
*
Needed coverage for that *addi_mul pattern. Committed.
gcc/testsuite:
* gcc.target/cris/biap-mul.c: New test.
---
gcc/testsuite/gcc.target/cris/biap-mul.c | 15 +++
1 file changed, 15 insertions(+)
create mode 100644 gcc/testsuite/gcc.target/cris/biap-mul.c
diff --git
Ever since the canonicalization clean-up of (mult X (1 << N)) into
(ashift X N) outside addresses, the CRIS addi patterns have been
unmatched. No big deal.
Unfortunately, nobody thought of adjusting reloaded addresses, so
transforming mult into a shift has to be a kludged for when reload
decides
Hi!
First off, thanbk you for the patch!
On Mon, Feb 15, 2021 at 11:22:52PM +, Neven Sajko via Gcc-patches wrote:
> There is a long-standing, but undocumented GCC inline assembly feature
> that's part of the extended asm GCC extension to C and C++: extended
> asm empty input constraints.
The intention of HONOR_REG_ALLOC_ORDER is to ensure that IRA allocates
registers in the order given by REG_ALLOC_ORDER. However in
ira_better_spill_reload_regno_p, there is still a place where the
calculation depends on the presence of REG_ALLOC_ORDER, ignoring
HONOR_REG_ALLOC_ORDER macro
The attached change adds mi_thunk support for vcalls on hppa. Tested on
hppa-unknown-linux-gnu,
hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11. Committed to trunk and gcc-10.
Dave
--
John David Anglin dave.ang...@bell.net
Add mi_thunk support for vcalls on hppa.
gcc/ChangeLog:
PR
The fix for 98741 introduced two issues. (a) recursive header units
iced because we tried to read the preprocessor state after having failed
to read the config. (b) we could have duplicate imports of named
modules in our pending queue, and that would lead to duplicate requests
for pathnames,
A member function can be defined in a different header-file than the one
defining the class. In such situations we must unmark the decl as
imported. When the entity is a template we failed to unmark the
template_decl.
Perhaps the duplication of these flags on the template_decl from the
On 15/02/21 23:22 +, Neven Sajko wrote:
There is a long-standing, but undocumented GCC inline assembly feature
that's part of the extended asm GCC extension to C and C++: extended
asm empty input constraints.
Although I don't really use extended asm much, and I never contributed
to GCC
On Fri, Feb 19, 2021 at 07:12:42PM +, Kwok Cheung Yeung wrote:
> I have opted for a union of completion_sem (for tasks that are undeferred)
> and a struct gomp_team *detach_team (for deferred tasks) that holds the team
> if the completion event has not yet fulfilled, or NULL if is it. I don't
Hi Alex,
On 22/02/2021 10:20, Alex Coplan wrote:
For the testcase, you might want to use the one I posted most recently:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98791#c3
which avoids the dependency on the aarch64-autovec-preference param
(which is in GCC 11 only) as this will simplify
Arnaud Charlet 于2021年2月18日周四 下午3:38写道：
>
> > For MIPS N64 and N32:
> > add GNATRTL_128BIT_PAIRS to LIBGNAT_TARGET_PAIRS
> > add GNATRTL_128BIT_OBJS to EXTRA_GNATRTL_NONTASKING_OBJS
> >
> > gcc/ada/ChangeLog:
> > PR ada/98996
> > * Makefile.rtl (LIBGNAT_TARGET_PAIRS,
On Mon, Feb 22, 2021 at 01:06:56PM +0100, Tobias Burnus wrote:
> Normal dummy arguments get some additional redirection if they are
> OPTIONAL; however, that's not the case for dummy procedures.
>
> That was shown by a simple 'procedure(), optional :: proc' example
> in the PR. – The fix is as
Normal dummy arguments get some additional redirection if they are
OPTIONAL; however, that's not the case for dummy procedures.
That was shown by a simple 'procedure(), optional :: proc' example
in the PR. – The fix is as simple.
However, I thought it still makes sense to test all combinations
PING^3
On 2/9/21 9:37 AM, Martin Liška wrote:
PING^2
@Honza: ?
On 1/29/21 2:33 PM, Martin Liška wrote:
PING^1
On 1/25/21 1:51 PM, Martin Liška wrote:
On 1/22/21 3:33 PM, Jan Hubicka wrote:
It is definitly doable (gcov machinery is quite flexible WRT having more
types of counters).
Yes,
Hi Andre,
Thanks for fixing this.
On 19/02/2021 10:53, Andre Vieira (lists) via Gcc-patches wrote:
> Hi,
>
> This patch makes sure that allocno copies are not created for unordered
> modes. The testcases in the PR highlighted a case where an allocno copy was
> being created for:
> (insn 121 120
On Fri, Feb 19, 2021 at 9:08 AM Alexandre Oliva wrote:
>
> Here's an improved version of the patch. Regstrapped on
> x86_64-linux-gnu, with and without a patchlet that moved multi-pieces
> ahead of setmem, and also tested with riscv32-elf.
>
> Is it ok to install? Or should it wait for stage1?
This adds another dump of the SLP subgraph we're throwing at costing.
Bootstrapped and tested on x86_64-unknown-linux-gnu, pushed.
2021-02-22 Richard Biener
* tree-vect-slp.c (vect_bb_vectorization_profitable_p): Dump
costed subgraph.
---
gcc/tree-vect-slp.c | 9 +
1
On Sat, Feb 20, 2021 at 11:49 AM Martin Liška wrote:
>
> After g:1a2a7096e5e20d736c6138179470b21aa5a74864 we forbid inlining
> for a VLA types. What we miss is setting inline_forbidden_reason
> variable.
>
> Fixes:
>
> ./xgcc -B. -O3 -c
>
This adds a missing accumulation to ret.
Bootstrapped and tested on x86_64-unknown-linux-gnu, pushed.
2021-02-22 Richard Biener
PR tree-optimization/99165
* gimple-ssa-store-merging.c (pass_store_merging::process_store):
Accumulate changed to ret.
*
On Fri, 19 Feb 2021, Tamar Christina wrote:
> Hi Richi,
>
> The attached testcase ICEs due to a couple of issues.
> In the testcase you have two SLP instances that share the majority of their
> definition with each other. One tree defines a COMPLEX_MUL sequence and the
> other tree a
60 matches
Mail list logo
