Hi!
When debugging PR112750, I've noticed some issues in the computation
of precisions and the following patch attempts to fix those.
The pass uses range_to_prec function, which possibly using ranger returns
minimum precision of some operand in the style that libgcc _BitInt
entrypoints expect,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112801
Richard Biener changed:
What|Removed |Added
Keywords||wrong-code
--- Comment #2 from
On Fri, 1 Dec 2023, Jakub Jelinek wrote:
> Hi!
>
> torture/bitint-39.c ICEs with -O1; the problem is that the
> finish_arith_overflow code in one spot replaces use_stmt with an
> assignment or cast, but if unlucky and m_gsi iterator is the same statement,
> when the code later
> tree
On Fri, 1 Dec 2023, Jakub Jelinek wrote:
> Hi!
>
> The .{ADD,SUB}_OVERFLOW lowering is implemented by performing normal
> addition/subtraction (perhaps extended to even more bits than normally by
> continuing with extended values of operands after running of normal bits)
> and in addition to
From: Pan Li
If we want to extract 64bit value but ELEN < 64, we use RVV
vector mode with EEW = 32 to extract the highpart and lowpart.
However, this approach doesn't honor DFmode when movdf pattern
when ZVE32f and of course results in ICE when zve32f.
This patch would like to reuse the
libphobos/ChangeLog:
* m4/druntime/cpu.m4: Support loongarch* targets.
* libdruntime/Makefile.am: Same.
* libdruntime/Makefile.in: Regenerate.
* configure: Regenerate.
---
libphobos/configure | 21 ++-
libphobos/libdruntime/Makefile.am | 3 +
This patchset is based on Zixing Liu's initial support patch:
https://gcc.gnu.org/pipermail/gcc-patches/2023-September/631260.html
Update v1 -> v2:
Rebased onto the dmd/druntime upstream state.
Regtested on loongarch64-linux-gnu with the following result:
=== libphobos Summary
libphobos/ChangeLog:
* src/std/math/hardware.d: Implement FP control.
* libdruntime/config/loongarch/switchcontext.S: New file.
---
.../config/loongarch/switchcontext.S | 133 ++
libphobos/src/std/math/hardware.d | 53 +++
2 files
gcc/ChangeLog:
* config/loongarch/loongarch-d.cc: Undefine LoongArch32.
Define LoongArch_SF, LoongArch_F32, LoongArch_F64
gcc/d/ChangeLog:
* dmd/cond.d: Same.
* implement-d.texi: Same.
---
gcc/config/loongarch/loongarch-d.cc | 27 ++-
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112793
Richard Biener changed:
What|Removed |Added
Status|NEW |ASSIGNED
Hi!
torture/bitint-39.c ICEs with -O1; the problem is that the
finish_arith_overflow code in one spot replaces use_stmt with an
assignment or cast, but if unlucky and m_gsi iterator is the same statement,
when the code later
tree clobber = build_clobber (TREE_TYPE (var), CLOBBER_EOL);
> -Original Message-
> From: Richard Biener
> Sent: Friday, December 1, 2023 3:04 PM
> To: Jiang, Haochen
> Cc: gcc-patches@gcc.gnu.org; Liu, Hongtao ;
> ubiz...@gmail.com
> Subject: Re: [PATCH] i386: Mark Xeon Phi ISAs as deprecated
>
> On Fri, Dec 1, 2023 at 3:22 AM Haochen Jiang
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112793
--- Comment #11 from Sam James ---
Created attachment 56747
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=56747=edit
another cvised testcase
Hi!
The .{ADD,SUB}_OVERFLOW lowering is implemented by performing normal
addition/subtraction (perhaps extended to even more bits than normally by
continuing with extended values of operands after running of normal bits)
and in addition to that trying to compute if certain sets of bits are either
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112788
Richard Biener changed:
What|Removed |Added
Priority|P3 |P1
CC|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112787
--- Comment #2 from Richard Biener ---
I think you should make sure to not use VLA vectors in lowering since lowering
doesn't really support using those (much similar to BB SLP).
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112786
Richard Biener changed:
What|Removed |Added
Target Milestone|--- |14.0
--- Comment #2 from Richard
On 11/30/23 16:45, Juergen Christ wrote:
> Commit 466b100e5fee808d77598e0f294654deec281150 introduced a bug in
> s390_md_asm_adjust if vector extensions are not available. Fix the control
> flow of this function to not adjust long double values.
>
> gcc/ChangeLog:
>
> *
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112753
--- Comment #3 from GCC Commits ---
The master branch has been updated by Andreas Krebbel :
https://gcc.gnu.org/g:8a2e428591dc97d573cf5c1789c84c8a3884b364
commit r14-6040-g8a2e428591dc97d573cf5c1789c84c8a3884b364
Author: Juergen Christ
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108476
Florian Weimer changed:
What|Removed |Added
CC||fw at gcc dot gnu.org
--- Comment #4
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112725
Jakub Jelinek changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112725
--- Comment #6 from GCC Commits ---
The master branch has been updated by Jakub Jelinek :
https://gcc.gnu.org/g:28944a016bb5eef9a99691b3c55b124882375180
commit r14-6039-g28944a016bb5eef9a99691b3c55b124882375180
Author: Jakub Jelinek
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96284
Bug 96284 depends on bug 91093, which changed state.
Bug 91093 Summary: Error on implicit int by default
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91093
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91093
Florian Weimer changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91092
Florian Weimer changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96284
Bug 96284 depends on bug 106416, which changed state.
Bug 106416 Summary: -Wint-conversion should be an error, not a pedwarn
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106416
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106416
Florian Weimer changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
On Thu, Nov 30, 2023 at 2:42 PM Xi Ruoyao wrote:
>
> Recently there are some people building GCC with srcdir == objdir and
> the attempts just failed [1]. So stop to say "it should work". OTOH
> objdir as a subdirectory of srcdir works: we've built GCC in LFS [2]
> and BLFS [3] this way for
On Thu, Nov 30, 2023 at 3:12 PM Richard Sandiford
wrote:
>
> Ping
OK.
Thanks,
Richard.
> Richard Sandiford writes:
> > This is a ping+rebase of the patch below. I've also optimised the
> > handling of ignored attributes so that we don't register empty tables.
> > There was also a typo in the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96284
--- Comment #12 from GCC Commits ---
The master branch has been updated by Florian Weimer :
https://gcc.gnu.org/g:4ecfa6c4a9df19b364cbdf6d9de5f0b98c22bd8f
commit r14-6036-g4ecfa6c4a9df19b364cbdf6d9de5f0b98c22bd8f
Author: Florian Weimer
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=44209
--- Comment #12 from GCC Commits ---
The master branch has been updated by Florian Weimer :
https://gcc.gnu.org/g:ff9efa3fc48baa1dee7ed376c25ecfcfbc28d35c
commit r14-6038-gff9efa3fc48baa1dee7ed376c25ecfcfbc28d35c
Author: Florian Weimer
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96284
--- Comment #13 from GCC Commits ---
The master branch has been updated by Florian Weimer :
https://gcc.gnu.org/g:9715c545d33b3a32ddc1ae817ba9356ade1fb9df
commit r14-6037-g9715c545d33b3a32ddc1ae817ba9356ade1fb9df
Author: Florian Weimer
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91093
--- Comment #4 from GCC Commits ---
The master branch has been updated by Florian Weimer :
https://gcc.gnu.org/g:4ee2aca7ca098cc6e622a9a41e63224af37e6532
commit r14-6034-g4ee2aca7ca098cc6e622a9a41e63224af37e6532
Author: Florian Weimer
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96284
--- Comment #10 from GCC Commits ---
The master branch has been updated by Florian Weimer :
https://gcc.gnu.org/g:55e94561e97ed0bce4774aa1c6b5d5d82209a379
commit r14-6033-g55e94561e97ed0bce4774aa1c6b5d5d82209a379
Author: Florian Weimer
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96284
--- Comment #11 from GCC Commits ---
The master branch has been updated by Florian Weimer :
https://gcc.gnu.org/g:4ee2aca7ca098cc6e622a9a41e63224af37e6532
commit r14-6034-g4ee2aca7ca098cc6e622a9a41e63224af37e6532
Author: Florian Weimer
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91092
--- Comment #22 from GCC Commits ---
The master branch has been updated by Florian Weimer :
https://gcc.gnu.org/g:55e94561e97ed0bce4774aa1c6b5d5d82209a379
commit r14-6033-g55e94561e97ed0bce4774aa1c6b5d5d82209a379
Author: Florian Weimer
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96284
--- Comment #9 from GCC Commits ---
The master branch has been updated by Florian Weimer :
https://gcc.gnu.org/g:2c3db94d9fd07681f8806dae93d555779ff4dfb1
commit r14-6032-g2c3db94d9fd07681f8806dae93d555779ff4dfb1
Author: Florian Weimer
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106416
--- Comment #6 from GCC Commits ---
The master branch has been updated by Florian Weimer :
https://gcc.gnu.org/g:2c3db94d9fd07681f8806dae93d555779ff4dfb1
commit r14-6032-g2c3db94d9fd07681f8806dae93d555779ff4dfb1
Author: Florian Weimer
Date:
On Fri, 1 Dec 2023, Hans-Peter Nilsson wrote:
> > From: Hans-Peter Nilsson
> > Date: Thu, 30 Nov 2023 18:09:10 +0100
>
> Richard B.:
> > > > In the end we might need to move/duplicate the test to some
> > > > gcc.target/* dir and restrict it to a specific tuning.
> >
> > I intend to post
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112789
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment
On Fri, Dec 1, 2023 at 3:22 AM Haochen Jiang wrote:
>
> Since Knight Landing and Knight Mill microarchitectures are EOL, we
> would like to remove its support in GCC 15. In GCC 14, we will first
> emit a warning for the usage.
I think it's better to keep supporting -mtune/arch=knl without
This patch leverages the same approach as vwcvt.
Before this patch:
.L5:
add a3,s0,s1
add a4,s6,s1
add a5,s7,s1
vsetvli zero,s0,e32,m4,ta,ma
vle32.v v16,0(s1)
vle32.v v12,0(a3)
mv s1,s2
vle32.v v8,0(a4)
On Thu, Nov 30, 2023 at 5:21 PM Sebastian Huber
wrote:
>
> In libgcov we use defined (__LIBGCC_HAVE_LIBATOMIC), so we must define it only
> if needed (vs. #if __LIBGCC_HAVE_LIBATOMIC).
For consistency wouldn't it be better to change the user side in libgcc?
> gcc/c-family/ChangeLog:
>
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110734
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112776
JuzheZhong changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
On 11/30/23 17:34, Jakub Jelinek wrote:
> On Wed, Nov 29, 2023 at 07:27:20PM +0100, Jakub Jelinek wrote:
>> Given that the s390 backend defines pretty much the same target hook
>> as rs6000, I believe it suffers (at least when using -mvx?) the same
>> problem as rs6000, though admittedly this is
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112776
--- Comment #1 from GCC Commits ---
The trunk branch has been updated by Lehua Ding :
https://gcc.gnu.org/g:923a67f17badcbe6e2b2e5d3570a265443258c8e
commit r14-6027-g923a67f17badcbe6e2b2e5d3570a265443258c8e
Author: Juzhe-Zhong
Date: Fri
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112778
Kewen Lin changed:
What|Removed |Added
CC||aoliva at gcc dot gnu.org,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112793
--- Comment #10 from Andrew Pinski ---
Created attachment 56746
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=56746=edit
Delta reduced testcase
This one is slightly less reduced than the other one but this is a delta based
reduced
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110734
--- Comment #18 from Julian Waters ---
Oops, I meant warning: 'no_reorder' attribute ignored [-Wattributes] in my
above comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110734
--- Comment #17 from Julian Waters ---
Looking at the source of the C++ parser it doesn't seem like asm (""); is
considered a statement but rather is correctly parsed as a declaration, see
cp_parser_block_declaration and
I ran into another issue while devising tests for redeclarations of
xobj member functions as static member functions and vice versa. I am
pretty sure by the literal wording of the standard, this is well formed.
template
concept Constrain = true;
struct S {
void f(this auto, Constrain auto) {};
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110734
--- Comment #16 from Xi Ruoyao ---
(In reply to Xi Ruoyao from comment #15)
> Alright, this is a C++ issue and I mistakenly assumed it was C.
Note that in C++ we have some inconsistency with the standard. In the standard
asm(...) are
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110734
Xi Ruoyao changed:
What|Removed |Added
Summary|Attributes cannot be|Attributes cannot be
Ping.
On Fri, 2023-11-24 at 17:09 +0800, Xi Ruoyao wrote:
> With -fno-fp-int-builtin-inexact, trunc is not allowed to raise
> FE_INEXACT and it should produce an integral result (if the input is not
> NaN or Inf). Thus FE_INEXACT should not be raised.
>
> But (int)x may raise FE_INEXACT when x
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110734
Xi Ruoyao changed:
What|Removed |Added
Status|NEW |WAITING
--- Comment #14 from Xi Ruoyao
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110734
Xi Ruoyao changed:
What|Removed |Added
Summary|Attributes cannot be|Attributes cannot be
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112764
--- Comment #10 from martin ---
Thanks for the speedy fix! I just thought about a variation, which should now
with the fix work as well (was not yet able to compile current dev branch, so
cannot check myself):
program assoc_ptr_in
implicit
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105118
康桓瑋 changed:
What|Removed |Added
Resolution|--- |INVALID
Status|SUSPENDED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112783
Sam James changed:
What|Removed |Added
See Also||https://github.com/Juniper/
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112803
Bug ID: 112803
Summary: : to(Args&&... args) is missing Mandates
Product: gcc
Version: 14.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110734
--- Comment #12 from Julian Waters ---
Will do, will save the new attribute for gcc 15 and just fix the attribute
specifier sequence here
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59874
Sam James changed:
What|Removed |Added
CC||sjames at gcc dot gnu.org
See
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112793
Andrew Pinski changed:
What|Removed |Added
Last reconfirmed||2023-12-01
Ever confirmed|0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112793
Andrew Pinski changed:
What|Removed |Added
Attachment #56744|0 |1
is obsolete|
Tested x86_64-pc-linux-gnu, applying to trunk.
-- 8< --
More adjustments to allow for explicit object parameters in lambdas. This
has no practical effect until that patch goes in, but applying this
separately seems reasonable.
gcc/cp/ChangeLog:
* semantics.cc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112786
Hans-Peter Nilsson changed:
What|Removed |Added
Keywords||patch
--- Comment #1 from
> From: Hans-Peter Nilsson
> Date: Thu, 30 Nov 2023 18:09:10 +0100
> I intend to post two alternative patches to get this
> resolved:
> 2: gcc.dg/tree-ssa/scev-[3-5].c skipped for arm*, xfailed
>only on h8300-*-* and ia32.
(Except as mentioned, the XPASS issue does not apply to
h8300; it's
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112802
Bug ID: 112802
Summary: : _ToClosure::operator() has no constraints
Product: gcc
Version: 14.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component:
> From: Hans-Peter Nilsson
> Date: Thu, 30 Nov 2023 18:09:10 +0100
Richard B.:
> > > In the end we might need to move/duplicate the test to some
> > > gcc.target/* dir and restrict it to a specific tuning.
>
> I intend to post two alternative patches to get this
> resolved:
> 1: Move the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112793
--- Comment #7 from Andrew Pinski ---
e.g. r[8 * 2] is r[16] and 16 >= 16 :).
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112793
--- Comment #6 from Andrew Pinski ---
(In reply to Sam James from comment #5)
> Created attachment 56744 [details]
> reduced.ii
Note this reduced testcase has undefined code in it ...
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112793
--- Comment #5 from Sam James ---
Created attachment 56744
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=56744=edit
reduced.ii
All regressions (zve64d/zvl128b/zvl256b/zvl512b/zvl1024b) passed.
juzhe.zh...@rivai.ai
From: Juzhe-Zhong
Date: 2023-12-01 08:51
To: gcc-patches
CC: kito.cheng; kito.cheng; jeffreyalaw; rdapp.gcc; Juzhe-Zhong
Subject: [PATCH] RISC-V: Fix VSETVL PASS regression
This patch fix 2 regression (one
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112801
--- Comment #1 from JuzheZhong ---
Known issue for a long time.
I realized that 2 years ago:
https://github.com/riscv-collab/riscv-gnu-toolchain/issues/1075
when I developed the first version of RVV GCC (rvv-next).
I don't think it is
Hi,
SImode in float register is supported on P7 above. It causes "fctiw"
can be generated on old 32-bit processors as the output operand of
fctiw insn is a SImode in float/double register. This patch fixes the
problem by adding an expand and an insn pattern for fctiw. The output
of new pattern
Hi,
The "fctid" is supported on 64-bit Power processors and powerpc 476. It
need a guard to check it. The patch fixes the issue.
Bootstrapped and tested on x86 and powerpc64-linux BE and LE with
no regressions. Is this OK for trunk?
Thanks
Gui Haochen
ChangeLog
rs6000: guard fctid on PPC64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112801
Bug ID: 112801
Summary: [14] RISC-V vector: failure to mask top bits during
type conversion
Product: gcc
Version: 14.0
Status: UNCONFIRMED
Severity: normal
> Hmm, I would suggest you put reg_needed into the class and accumulate
> over all vec_construct, with your patch you pessimize a single v32qi
> over two separate v16qi for example. Also currently the whole block is
> gated with INTEGRAL_TYPE_P but register pressure would be also
> a concern for
> From: Hans-Peter Nilsson
> Date: Thu, 30 Nov 2023 18:09:10 +0100
> I intend to post two alternative patches to get this
> resolved:
> 1: Move the tests to gcc.target/i386/scev-[3-5].c
> 2: gcc.dg/tree-ssa/scev-[3-5].c skipped for arm*, xfailed
>only on h8300-*-* and ia32.
Correction:
Since Knight Landing and Knight Mill microarchitectures are EOL, we
would like to remove its support in GCC 15. In GCC 14, we will first
emit a warning for the usage.
gcc/ChangeLog:
* config/i386/driver-i386.cc (host_detect_local_cpu):
Do not append "-mno-" for Xeon Phi ISAs.
Hi all,
Since Knight Landing and Knight Mill microarchitectures were EOL in 2019
and previously ICC and ICX has removed the support and emitted errors, we
would also like to remove the support in GCC to reduce maintainence effort.
The deprecated Xeon Phi ISAs are AVX512PF, AVX512ER, AVX5124VNNIW,
On Thu, 2023-11-30 at 08:44 -0700, Jeff Law wrote:
>
>
> On 11/29/23 02:33, Xi Ruoyao wrote:
> > On Mon, 2023-11-27 at 23:06 -0700, Jeff Law wrote:
> > > This has (of course) been tested on rv64. It's also been bootstrapped
> > > and regression tested on x86. Bootstrap and regression tested (C
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112773
--- Comment #7 from JuzheZhong ---
Here is the compiler explorer for reference and easily see the issue:
https://godbolt.org/z/8v1dsKG3f
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112773
--- Comment #6 from Patrick O'Neill ---
(In reply to JuzheZhong from comment #3)
> I see. I didn't reproduce the ICE since I didn't enable gcc_assert_checking.
>
> Could you tell me how to enable it ?
I didn't explicitly enable checking. The
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112773
--- Comment #5 from JuzheZhong ---
Besides, the mask type is:
vector([4,4])
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112773
--- Comment #4 from JuzheZhong ---
After reproducing the ICE.
I confirm the root cause is because we enable vec_extract for mask:
;; -
;; This extracts a bit (via QImode)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112773
--- Comment #3 from JuzheZhong ---
I see. I didn't reproduce the ICE since I didn't enable gcc_assert_checking.
Could you tell me how to enable it ?
I hack the codes as follows:
diff --git a/gcc/poly-int.h b/gcc/poly-int.h
index
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112640
Sam James changed:
What|Removed |Added
Status|UNCONFIRMED |WAITING
Last reconfirmed|
> Date: Sun, 19 Nov 2023 17:47:56 -0700
> From: Jeff Law
> Locally we have had this enabled at -O1 and above to encourage testing,
> but I'm thinking that for the trunk enabling at -O2 and above is the
> right thing to do.
Yes.
> Thoughts, comments, recommendations?
Sounds great!
It'd be
Ah. I see:
--- a/gcc/config/riscv/riscv.md
+++ b/gcc/config/riscv/riscv.md
@@ -2336,9 +2336,7 @@ (define_expand "cpymem"
(use (match_operand:SI 3 "const_int_operand"))])]
""
{
- if (riscv_vector::expand_block_move (operands[0], operands[1], operands[2]))
-DONE;
- else if
Hi, Robin.
Thanks for working on this. I know this is a tedious work.
A couple comments here:
- if (TARGET_ZBB || TARGET_XTHEADBB)
+ if (TARGET_VECTOR && stringop_strategy & STRINGOP_STRATEGY_VECTOR)
+{
+ bool ok = riscv_vector::expand_strcmp (result, src1, src2, bytes_rtx,
+
This patch fix 2 regression (one is bug regression, the other is performance
regression).
Those 2 regressions are both we are comparing ratio for same AVL in wrong place.
1. BUG regression:
avl_single-84.c:
f0:
li a5,999424
add a1,a1,a5
li a4,299008
On 11/30/23 15:22, Robin Dapp wrote:
Hi,
this adds vectorized implementations of strcmp and strncmp as well as
strlen. strlen falls back to the previously implemented rawmemchr.
Also, it fixes a rawmemchr bug causing a SPEC2017 execution failure:
We would only ever increment the source
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112798
--- Comment #1 from Patrick O'Neill ---
See also:
rv64gcv_zve64d: pr112797
rv64gcv_zve32f: pr112798
rv32gcv_zve64d: pr112799
rv32gcv_zve32f: pr112800
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112799
--- Comment #1 from Patrick O'Neill ---
See also:
rv64gcv_zve64d: pr112797
rv64gcv_zve32f: pr112798
rv32gcv_zve64d: pr112799
rv32gcv_zve32f: pr112800
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112800
--- Comment #1 from Patrick O'Neill ---
See also:
rv64gcv_zve64d: pr112797
rv64gcv_zve32f: pr112798
rv32gcv_zve64d: pr112799
rv32gcv_zve32f: pr112800
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112797
--- Comment #1 from Patrick O'Neill ---
See also:
rv64gcv_zve64d: pr112797
rv64gcv_zve32f: pr112798
rv32gcv_zve64d: pr112799
rv32gcv_zve32f: pr112800
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112800
Bug ID: 112800
Summary: RISC-V regression testsuite errors with rv32gcv_zve32f
Product: gcc
Version: 14.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112799
Bug ID: 112799
Summary: RISC-V regression testsuite errors with rv32gcv_zve64d
Product: gcc
Version: 14.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
1 - 100 of 369 matches
Mail list logo