PATCH: alignment in .string tesdt

Marcus Meissner meissner at suse.de
Mon Aug 19 02:32:55 CDT 2002


Hi,

The .string test is wrong regards wise for CPUs which check on instruction
alignment. You cannot just emit a string into the flow of instructions.

Ciao, Marcus

License: LGPL
Changelog:
	Do not emit .string into the instruction stream, but into
	the .data section (to avoid alignment assertions).

Index: configure.ac
===================================================================
RCS file: /home/wine/wine/configure.ac,v
retrieving revision 1.70
diff -u -r1.70 configure.ac
--- configure.ac	15 Aug 2002 22:08:41 -0000	1.70
+++ configure.ac	19 Aug 2002 07:31:22 -0000
@@ -743,7 +743,11 @@
 dnl **** Check for .string in assembler ****
 
 AC_CACHE_CHECK([whether assembler accepts .string], ac_cv_c_asm_string,
-    WINE_TRY_ASM_LINK([".string \"test\""],,,ac_cv_c_asm_string="yes",ac_cv_c_asm_string="no"))
+    WINE_TRY_ASM_LINK([
+    ".data\n"
+    ".string \"test\"\n"
+    ".text\n"
+    ],,,ac_cv_c_asm_string="yes",ac_cv_c_asm_string="no"))
 if test "$ac_cv_c_asm_string" = "yes"
 then
   AC_DEFINE(HAVE_ASM_STRING, 1, [Define to use .string instead of .ascii])



More information about the wine-patches mailing list