Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: doug.gilmore at imgtec dot com
Target Milestone: ---
Created attachment 37614
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37614&action=edit
extracted daxpy
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69710
--- Comment #1 from Doug Gilmore ---
Created attachment 37615
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37615&action=edit
daxpy for DP (previous was for SP)
Compilation example:
arm-linux-gnueabihf-gcc -O3 -save-temps daxpy.c saxpy.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69710
--- Comment #5 from Doug Gilmore ---
Thanks for checking on AArch64 Andrew.
BTW, I made my (incorrect) hunch by running a test on gcc113, where
the installed 4.8 compile showed problems for both DP and SP. (I
assumed that the problem was addres
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69710
--- Comment #10 from Doug Gilmore ---
Created attachment 37681
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37681&action=edit
prototype fix
> 1) we failed recognize that use 0 and 2 are identical to each other.
> This is because vectoriz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69710
--- Comment #12 from Doug Gilmore ---
> Yes, I proposed some cleanup passess after vectorization but richi
> thinks it's genrally expensive. So what's implmentation complexity
> of pass_dominator?
One thing we might consider is only enable it wh
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69710
--- Comment #13 from Doug Gilmore ---
I think this should be fairly straightforward to fix in the
autovectorization pass. Hopefully I should be able to post a patch
in the next few days.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69710
--- Comment #15 from Doug Gilmore ---
> I had a patch too, will send it for review in GCC7 if it's still needed.
Sorry I got side track last week and didn't make much progress.
Please go ahead and submit if you have something you feel comfortabl
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81025
--- Comment #1 from Doug Gilmore ---
Created attachment 41510
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41510&action=edit
Patch to constrain the number of multi-lib variants
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81025
--- Comment #2 from Doug Gilmore ---
Created attachment 41511
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41511&action=edit
patch needed to build r248863 for MIPS
: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: doug.gilmore at imgtec dot com
Target Milestone: ---
Created attachment 41509
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41509&action=edit
CPP output file
Our ToT GLIBC soft-float builds are
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81025
--- Comment #3 from Doug Gilmore ---
It appears that r248863 just tickles the bug. With
the attached example produced by delta the failure mode
is exposed by r248862.With luck, I may be able to
bisect the problem to an earlier commit.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81025
Doug Gilmore changed:
What|Removed |Added
CC||rguenth at gcc dot gnu.org
--- Comment #4
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81025
Doug Gilmore changed:
What|Removed |Added
Summary|[MIPS] soft-float glibc |[8 Regression] gcc ICE
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81025
--- Comment #9 from Doug Gilmore ---
> I bet this is a bug in reorg.c. It is the least used code (major
> target usage: MIPS and sparc only) and also one of the more buggy
> code.
You're right, compiling with -fno-delayed-branch doesn't tickle t
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78176
Doug Gilmore changed:
What|Removed |Added
CC||doug.gilmore at imgtec dot com
Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: doug.gilmore at imgtec dot com
Target Milestone: ---
It appears that r244397 introduces pealing for DP daxpy, which per
bug 69710, introduces a performance degradation
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78176
--- Comment #15 from Doug Gilmore ---
Created attachment 40631
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40631&action=edit
Prototype change to backout r216501.
> Bisected the problem to commit r216501:
The review discussion of r21650
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78176
--- Comment #16 from Doug Gilmore ---
Created attachment 40632
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40632&action=edit
Tweak to adjust_setup_cost (r220473).
Second patch associated with previous comment.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78176
--- Comment #17 from Doug Gilmore ---
> This really throws off the costing of substituting different IVs on
> MIPS.
I forgot to mention that for MIPS the net of effect r216501 is to not
produce indexed memory OPs in simple examples where we shoul
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78176
Doug Gilmore changed:
What|Removed |Added
CC||law at redhat dot com,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78176
--- Comment #20 from Doug Gilmore ---
I'll collect more tracing data on the costing problem.
Hopefully I post an update in the next few days.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79291
--- Comment #4 from Doug Gilmore ---
> It also looks like mips lacks implementation of any of the
> vectorizer cost hooks and thus defaults to
> default_builtin_vectorization_cost which means that unaligned
> loads/stores have double cost.
I have
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79291
--- Comment #5 from Doug Gilmore ---
> Bin: I suspect this is also now broken on ARM, can
> you check?
Oops, sorry I forgot that this problem is not exposed
on the original ARM/Neon for DP. Sorry for the noise.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79291
--- Comment #6 from Doug Gilmore ---
> It also looks like mips lacks implementation of any of the
> vectorizer cost hooks and thus defaults to
> default_builtin_vectorization_cost which means that unaligned
> loads/stores have double cost.
Removi
Assignee: unassigned at gcc dot gnu.org
Reporter: doug.gilmore at imgtec dot com
Target Milestone: ---
Created attachment 40920
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40920&action=edit
CPP output file for mips-mti-linux-gnu target
See:
https://sourceware
Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: doug.gilmore at imgtec dot com
Target Milestone: ---
Compiling the test example:
void daxpy(int n, double da, double * __restrict dx, double * __restrict dy
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77654
--- Comment #1 from Doug Gilmore ---
Created attachment 39651
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=39651&action=edit
Additional tracing used to identify problem.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77654
--- Comment #2 from Doug Gilmore ---
Created attachment 39652
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=39652&action=edit
Prototype fix for bug.
Severity: normal
Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: doug.gilmore at imgtec dot com
Target Milestone: ---
Reported in:
https://gcc.gnu.org/ml/gcc-patches/2016-09/msg02285.html
This issue was not
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72850
Doug Gilmore changed:
What|Removed |Added
CC||doug.gilmore at imgtec dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77808
Doug Gilmore changed:
What|Removed |Added
CC||clyon at gcc dot gnu.org
--- Comment #2 f
Priority: P3
Component: c
Assignee: unassigned at gcc dot gnu.org
Reporter: doug.gilmore at imgtec dot com
Created attachment 33440
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33440&action=edit
test example
I noticed that MultiSource/Benchmar
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63148
Doug Gilmore changed:
What|Removed |Added
CC||rguenther at suse dot de
--- Comment #2 f
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63148
--- Comment #6 from Doug Gilmore ---
> The input to the vectorizer is already bogus:
>
> _12 = i.0_5 + 536870911;
> _13 = global_data.b[_12];
Note that gimple out generated by the front end
is already problematic:
Before r187042:
D.1747 =
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63148
Doug Gilmore changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
++
Assignee: unassigned at gcc dot gnu.org
Reporter: doug.gilmore at imgtec dot com
Created attachment 33616
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33616&action=edit
test program
The attached test program fails with 4.7 up to ToT at -O2 on both x86
(I built x86_64 w
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63412
--- Comment #1 from Doug Gilmore ---
Created attachment 33617
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33617&action=edit
Modified version where type casts are modified.
Severity: normal
Priority: P3
Component: c
Assignee: unassigned at gcc dot gnu.org
Reporter: doug.gilmore at imgtec dot com
Target Milestone: ---
The commit r225260 broke the builds of the mips-{mti,img}-linux-gnu tool
chains.
To reproduce the problem, configure
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66747
--- Comment #4 from Doug Gilmore ---
Thanks!
I started up a build with the patch and it got through
the initial_gcc build so that is a good sign.
I'll send an update once the build is done.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66747
Doug Gilmore changed:
What|Removed |Added
CC||matthew.fortune at imgtec dot
com
--- Co
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66747
--- Comment #9 from Doug Gilmore ---
Our nightly builds are now clean with this patch.
Thanks!
41 matches
Mail list logo
