[Valgrind-developers] valgrind: r7965 - branches/OTRACK_BY_INSTRUMENTATION/coregrind/m_scheduler

[<sv...@va...>]

Author: sewardj
Date: 2008-04-30 11:01:25 +0100 (Wed, 30 Apr 2008)
New Revision: 7965

Log:
do_pre_run_checks: Add (even) more paranoia for alignment of ppc32/64
vector regs.



Modified:
   branches/OTRACK_BY_INSTRUMENTATION/coregrind/m_scheduler/scheduler.c


Modified: branches/OTRACK_BY_INSTRUMENTATION/coregrind/m_scheduler/scheduler.c
===================================================================
--- branches/OTRACK_BY_INSTRUMENTATION/coregrind/m_scheduler/scheduler.c	2008-04-30 09:41:32 UTC (rev 7964)
+++ branches/OTRACK_BY_INSTRUMENTATION/coregrind/m_scheduler/scheduler.c	2008-04-30 10:01:25 UTC (rev 7965)
@@ -568,10 +568,14 @@
 
 #  if defined(VGA_ppc32) || defined(VGA_ppc64)
    /* ppc guest_state vector regs must be 16 byte aligned for
-      loads/stores */
+      loads/stores.  This is important! */
    vg_assert(VG_IS_16_ALIGNED(& tst->arch.vex.guest_VR0));
    vg_assert(VG_IS_16_ALIGNED(& tst->arch.vex_shadow1.guest_VR0));
    vg_assert(VG_IS_16_ALIGNED(& tst->arch.vex_shadow2.guest_VR0));
+   /* be extra paranoid .. */
+   vg_assert(VG_IS_16_ALIGNED(& tst->arch.vex.guest_VR1));
+   vg_assert(VG_IS_16_ALIGNED(& tst->arch.vex_shadow1.guest_VR1));
+   vg_assert(VG_IS_16_ALIGNED(& tst->arch.vex_shadow2.guest_VR1));
 #  endif   
 }