[valgrind] [Bug 385843] [PATCH] ARM: mark caller-save VFP registes as trashed by calls

2017-10-17 Thread Ivo Raisr
https://bugs.kde.org/show_bug.cgi?id=385843 Ivo Raisr changed: What|Removed |Added Ever confirmed|0 |1 Status|UNCONFIRMED

[valgrind] [Bug 385843] [PATCH] ARM: mark caller-save VFP registes as trashed by calls

2017-10-17 Thread Sindre Aamås
https://bugs.kde.org/show_bug.cgi?id=385843 --- Comment #10 from Sindre Aamås --- The reason why I ran across this is likely that our toolchain has -mfpu=neon as default, which may be uncommon, which would explain why this has not been reported previously. -- You are

[valgrind] [Bug 385843] [PATCH] ARM: mark caller-save VFP registes as trashed by calls

2017-10-17 Thread Ivo Raisr
https://bugs.kde.org/show_bug.cgi?id=385843 --- Comment #9 from Ivo Raisr --- (In reply to Sindre Aamås from comment #6) > I have not looked at the stats, but the output is as follows. As suspected, there is a code bloat in the generated code because of the additional spilling

[valgrind] [Bug 385843] [PATCH] ARM: mark caller-save VFP registes as trashed by calls

2017-10-17 Thread Sindre Aamås
https://bugs.kde.org/show_bug.cgi?id=385843 --- Comment #8 from Sindre Aamås --- Created attachment 108398 --> https://bugs.kde.org/attachment.cgi?id=108398=edit Stats after -- You are receiving this mail because: You are watching all bug changes.

[valgrind] [Bug 385843] [PATCH] ARM: mark caller-save VFP registes as trashed by calls

2017-10-17 Thread Sindre Aamås
https://bugs.kde.org/show_bug.cgi?id=385843 --- Comment #7 from Sindre Aamås --- Created attachment 108397 --> https://bugs.kde.org/attachment.cgi?id=108397=edit Stats before -- You are receiving this mail because: You are watching all bug changes.

[valgrind] [Bug 385843] [PATCH] ARM: mark caller-save VFP registes as trashed by calls

2017-10-17 Thread Sindre Aamås
https://bugs.kde.org/show_bug.cgi?id=385843 --- Comment #6 from Sindre Aamås --- Q registers should be mostly relevant for NEON (often intrinsics or assembly routines with limited control flow). I ran memcheck with and without the change on a fairly NEON-intensive component

[valgrind] [Bug 385843] [PATCH] ARM: mark caller-save VFP registes as trashed by calls

2017-10-16 Thread Ivo Raisr
https://bugs.kde.org/show_bug.cgi?id=385843 --- Comment #5 from Ivo Raisr --- Your fix touches register allocation. It is crucial that both ARMInstr_Call() and getRRegUniverse_ARM() are kept in sync (hinted in getRRegUniverse_ARM as well) and that register allocator is

[valgrind] [Bug 385843] [PATCH] ARM: mark caller-save VFP registes as trashed by calls

2017-10-16 Thread Sindre Aamås
https://bugs.kde.org/show_bug.cgi?id=385843 Sindre Aamås changed: What|Removed |Added Attachment #108392|0 |1 is obsolete|

[valgrind] [Bug 385843] [PATCH] ARM: mark caller-save VFP registes as trashed by calls

2017-10-16 Thread Sindre Aamås
https://bugs.kde.org/show_bug.cgi?id=385843 --- Comment #3 from Sindre Aamås --- (In reply to Ivo Raisr from comment #2) > The comment in getRRegUniverse_ARM() should be more explicit about which > registers are caller save and callee save. > > And if this is indeed the case,

[valgrind] [Bug 385843] [PATCH] ARM: mark caller-save VFP registes as trashed by calls

2017-10-16 Thread Ivo Raisr
https://bugs.kde.org/show_bug.cgi?id=385843 Ivo Raisr changed: What|Removed |Added CC||iv...@ivosh.net --- Comment #2

[valgrind] [Bug 385843] [PATCH] ARM: mark caller-save VFP registes as trashed by calls

2017-10-16 Thread Sindre Aamås
https://bugs.kde.org/show_bug.cgi?id=385843 --- Comment #1 from Sindre Aamås --- Created attachment 108392 --> https://bugs.kde.org/attachment.cgi?id=108392=edit [PATCH] VEX/ARM: mark caller-save VFP registes as trashed by calls -- You are receiving this mail because: You