|
From: <sv...@va...> - 2012-08-03 03:21:50
|
florian 2012-08-03 04:21:42 +0100 (Fri, 03 Aug 2012)
New Revision: 12818
Log:
Comment only change.
Modified files:
trunk/include/valgrind.h
Modified: trunk/include/valgrind.h (+12 -2)
===================================================================
--- trunk/include/valgrind.h 2012-08-02 23:08:53 +01:00 (rev 12817)
+++ trunk/include/valgrind.h 2012-08-03 04:21:42 +01:00 (rev 12818)
@@ -3346,9 +3346,11 @@
# define VALGRIND_CFI_EPILOGUE
#endif
+/* Nb: On s390 the stack pointer is properly aligned *at all times*
+ according to the s390 GCC maintainer. (The ABI specification is not
+ precise in this regard.) Therefore, VALGRIND_ALIGN_STACK and
+ VALGRIND_RESTORE_STACK are not defined here. */
-
-
/* These regs are trashed by the hidden call. Note that we overwrite
r14 in s390_irgen_noredir (VEX/priv/guest_s390_irgen.c) to give the
function a proper return address. All others are ABI defined call
@@ -3356,6 +3358,14 @@
#define __CALLER_SAVED_REGS "0","1","2","3","4","5","14", \
"f0","f1","f2","f3","f4","f5","f6","f7"
+/* Nb: Although r11 is modified in the asm snippets below (inside
+ VALGRIND_CFI_PROLOGUE) it is not listed in the clobber section, for
+ two reasons:
+ (1) r11 is restored in VALGRIND_CFI_EPILOGUE, so effectively it is not
+ modified
+ (2) GCC will complain that r11 cannot appear inside a clobber section,
+ when compiled with -O -fno-omit-frame-pointer
+ */
#define CALL_FN_W_v(lval, orig) \
do { \
|