I noticed what looks like a typo in the code for the powerpc altivec register
set save/restore logic. I found the bug in both 3.10.1 and 3.9.
A suggested patch is shown below.
Regards,
Dave
=====================================
Fix typo saving altivec register v24
Fix a typo that saved vector register 25 in the slot reserved for
register 24. Verified by examining the code that restores v24 from
the same 16 byte offset (112) at label LafterFP9.
Signed-off-by: Dave Lerner <dlernerdr...@gmail.com>
diff --git a/coregrind/m_dispatch/dispatch-ppc32-linux.S
b/coregrind/m_dispatch/dispatch-ppc32-linux.S
index 6457219..3c246ef 100644
--- a/coregrind/m_dispatch/dispatch-ppc32-linux.S
+++ b/coregrind/m_dispatch/dispatch-ppc32-linux.S
@@ -158,7 +158,7 @@ LafterFP1:
li 6,128
stvx 25,6,1
li 6,112
- stvx 25,6,1
+ stvx 24,6,1
li 6,96
stvx 23,6,1
li 6,80
=================================
------------------------------------------------------------------------------
BPM Camp - Free Virtual Workshop May 6th at 10am PDT/1PM EDT
Develop your own process in accordance with the BPMN 2 standard
Learn Process modeling best practices with Bonita BPM through live exercises
http://www.bonitasoft.com/be-part-of-it/events/bpm-camp-virtual- event?utm_
source=Sourceforge_BPM_Camp_5_6_15&utm_medium=email&utm_campaign=VA_SF
_______________________________________________
Valgrind-users mailing list
Valgrind-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/valgrind-users
