https://bugs.kde.org/show_bug.cgi?id=368529
--- Comment #17 from Elliott Hughes ---
(/me checks check-in comment from
https://android-review.googlesource.com/393156...)
yeah, looks like i misremembered: it was SSE4 that was missing. from the bug
that links to, it was
https://bugs.kde.org/show_bug.cgi?id=368529
--- Comment #16 from Julian Seward ---
(In reply to Elliott Hughes from comment #15)
> x86 is useless because valgrind doesn't
> support SSE2 for x86 so you can't get as far as calling main,
Valgrind supports up to and including SSSE3
https://bugs.kde.org/show_bug.cgi?id=368529
--- Comment #15 from Elliott Hughes ---
fwiw, here's the current diff to be able to build valgrind 3.13.0 for Android
(arm,aarch64,x86-64). (x86 is useless because valgrind doesn't support SSE2 for
x86 so you can't get as far as
https://bugs.kde.org/show_bug.cgi?id=368529
--- Comment #14 from Elliott Hughes ---
> Redone, using the house conditionalisation scheme, in r16386. Please test.
(sorry, didn't have time to look until now.)
as i explained already, that's wrong.
anywhere you have this idiom in
https://bugs.kde.org/show_bug.cgi?id=368529
--- Comment #13 from Julian Seward ---
Redone, using the house conditionalisation scheme, in r16386. Please test.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.kde.org/show_bug.cgi?id=368529
Julian Seward changed:
What|Removed |Added
CC||jsew...@acm.org
--- Comment
https://bugs.kde.org/show_bug.cgi?id=368529
Ivo Raisr changed:
What|Removed |Added
Resolution|--- |FIXED
Status|ASSIGNED
https://bugs.kde.org/show_bug.cgi?id=368529
--- Comment #10 from Elliott Hughes ---
yeah, looking through the current clusters of #if..._android_linux, they're all
either wrong or the subject of an outstanding patch :-/
i've raised https://bugs.kde.org/show_bug.cgi?id=379878
https://bugs.kde.org/show_bug.cgi?id=368529
--- Comment #9 from Elliott Hughes ---
Created attachment 105569
--> https://bugs.kde.org/attachment.cgi?id=105569=edit
patch for https://bugs.kde.org/show_bug.cgi?id=368529#c8
--
You are receiving this mail because:
You are
https://bugs.kde.org/show_bug.cgi?id=368529
--- Comment #8 from Elliott Hughes ---
(In reply to Ivo Raisr from comment #7)
> I am a little worried about this condition in the patch:
>
> #if defined(ANDROID) && defined(__clang__)
>
> Nowhere in Valgrind sources we currently
https://bugs.kde.org/show_bug.cgi?id=368529
Ivo Raisr changed:
What|Removed |Added
Status|CONFIRMED |ASSIGNED
--- Comment #7 from Ivo
https://bugs.kde.org/show_bug.cgi?id=368529
Ivo Raisr changed:
What|Removed |Added
Assignee|jsew...@acm.org |iv...@ivosh.net
https://bugs.kde.org/show_bug.cgi?id=368529
--- Comment #5 from Elliott Hughes ---
this patch is still necessary with 3.12 (we updated AOSP recently). it's one of
only four issues building valgrind 3.12 out of the box for Android. worth
taking for 3.13?
--
You are receiving
https://bugs.kde.org/show_bug.cgi?id=368529
Elliott Hughes changed:
What|Removed |Added
CC||e...@google.com
--
You are
https://bugs.kde.org/show_bug.cgi?id=368529
--- Comment #4 from c...@google.com ---
Created attachment 105101
--> https://bugs.kde.org/attachment.cgi?id=105101=edit
diff to m_main.c
This is from https://android-review.googlesource.com/#/c/272640/
--
You are receiving this mail because:
You
https://bugs.kde.org/show_bug.cgi?id=368529
Mark Wielaard changed:
What|Removed |Added
CC||m...@klomp.org
--- Comment #3
https://bugs.kde.org/show_bug.cgi?id=368529
--- Comment #2 from c...@google.com ---
Julian, this was fixed for Android in
https://android-review.git.corp.google.com/#/c/272640
Could you review and take that patch?
Thanks.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.kde.org/show_bug.cgi?id=368529
--- Comment #1 from c...@google.com ---
The undefined symbols were due to the use of builtin functions
like __aeabi_memcpy* and __aeabi_memclr* from clang/llvm compiler.
Those symbols pulled in other modules in Android libc.a and then
require atexit and
https://bugs.kde.org/show_bug.cgi?id=368529
c...@google.com changed:
What|Removed |Added
CC||dimi...@google.com
--
You are receiving this
19 matches
Mail list logo