https://bugs.kde.org/show_bug.cgi?id=385411
Andreas Arnez changed:
What|Removed |Added
Resolution|--- |FIXED
Status|REPORTED
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #17 from Andreas Arnez ---
Created attachment 116557
--> https://bugs.kde.org/attachment.cgi?id=116557=edit
Address feedback for vector FP patch
Another delta-patch that addresses the remaining aspects from comment #12. The
vassert()
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #16 from Andreas Arnez ---
Created attachment 116556
--> https://bugs.kde.org/attachment.cgi?id=116556=edit
Change unsigned int to UInt
This is a delta-patch that changes the bit fields members from unsigned int to
UInt. This patch only
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #15 from Andreas Arnez ---
(In reply to Julian Seward from comment #12)
> (In reply to Andreas Arnez from comment #10)
> [...]
> + unsigned int v2 : 4;
> etc
>
> For new structs like this, I'd prefer you use the house type "UInt"
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #14 from Julian Seward ---
(In reply to Andreas Arnez from comment #9)
> For the record, Julian Seward commented the following in IRC:
>
> * Regarding the fused multiply-add/subs:
>
> "I think the *right* fix here is to create new Iops,
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #13 from Julian Seward ---
(In reply to Andreas Arnez from comment #11)
> Created attachment 116469 [details]
> Tests for vector FP support (v2)
>
> Revised version of the tests for the vector FP support. This version
> addresses the
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #12 from Julian Seward ---
(In reply to Andreas Arnez from comment #10)
> Created attachment 116468 [details]
> Vector floating point support (v2)
>
> Revised version of vector FP support. This should address all the feedback
> from
https://bugs.kde.org/show_bug.cgi?id=385411
Andreas Arnez changed:
What|Removed |Added
Attachment #115426|0 |1
is obsolete|
https://bugs.kde.org/show_bug.cgi?id=385411
Andreas Arnez changed:
What|Removed |Added
Attachment #115427|0 |1
is obsolete|
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #10 from Andreas Arnez ---
Created attachment 116468
--> https://bugs.kde.org/attachment.cgi?id=116468=edit
Vector floating point support (v2)
Revised version of vector FP support. This should address all the feedback
from comment #7
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #9 from Andreas Arnez ---
For the record, Julian Seward commented the following in IRC:
* Regarding the fused multiply-add/subs:
"I think the *right* fix here is to create new Iops, Iop_MAddF64x2 and
IOpMSubF64x2 and use those instead
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #8 from Andreas Arnez ---
(In reply to Vadim Barkov from comment #2)
> Created attachment 115426 [details]
> z13 vector floating point support (tests)
Here are some more comments about the test cases:
> [...]
> ---
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #7 from Andreas Arnez ---
(In reply to Vadim Barkov from comment #6)
> Good news! Is any additional work needed on this issue?
As I said before, long lines should be avoided. And there are a few other nits
in the implementation (attachment
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #6 from Vadim Barkov ---
(In reply to Andreas Arnez from comment #5)
Good news! Is any additional work needed on this issue?
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #5 from Andreas Arnez ---
(In reply to Andreas Arnez from comment #4)
> [...] I have encountered one potential problem, which
> may or may not be caused by your changes. I'll look further into it and let
> you know when I know more.
OK, I
https://bugs.kde.org/show_bug.cgi?id=385411
Julian Seward changed:
What|Removed |Added
Attachment #115427|0 |1
is patch|
https://bugs.kde.org/show_bug.cgi?id=385411
Julian Seward changed:
What|Removed |Added
Attachment #115426|application/mbox|text/plain
mime type|
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #4 from Andreas Arnez ---
(In reply to Vadim Barkov from comment #3)
> Created attachment 115427 [details]
> z13 vector floating point support (code)
>
> Please take a look
Thanks, this looks pretty good already. One thing I'd like to
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #3 from Vadim Barkov ---
Created attachment 115427
--> https://bugs.kde.org/attachment.cgi?id=115427=edit
z13 vector floating point support (code)
Please take a look
--
You are receiving this mail because:
You are watching all bug
https://bugs.kde.org/show_bug.cgi?id=385411
--- Comment #2 from Vadim Barkov ---
Created attachment 115426
--> https://bugs.kde.org/attachment.cgi?id=115426=edit
z13 vector floating point support (tests)
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.kde.org/show_bug.cgi?id=385411
Vadim Barkov changed:
What|Removed |Added
CC||vbr...@gmail.com
--- Comment #1 from Vadim
https://bugs.kde.org/show_bug.cgi?id=385411
Florian Weimer changed:
What|Removed |Added
CC||fwei...@redhat.com
--
You are receiving this
https://bugs.kde.org/show_bug.cgi?id=385411
Andreas Arnez changed:
What|Removed |Added
Blocks|366413 |
Referenced Bugs:
https://bugs.kde.org/show_bug.cgi?id=385411
Andreas Arnez changed:
What|Removed |Added
Blocks||366413
Referenced
24 matches
Mail list logo