NonSFI mode: Make tests/common/register_set.h PIC-friendly.

tests/common/register_set.h

Index: tests/common/register_set.h
diff --git a/tests/common/register_set.h b/tests/common/register_set.h
index 5bb747d174b1c8fa1d02e322504c2c2d95fbf7ee..f01e9f29cb7ef746f4ff1f8874e558a8a680a22c 100644
--- a/tests/common/register_set.h
+++ b/tests/common/register_set.h
@@ -240,7 +240,7 @@ extern const uint8_t kX86FlagBits[5];
         #def_func ":\n" \
         /* Push most of "struct NaClSignalContext" in reverse order. */ \
         "push $0\n"  /* Leave space for flags */ \
-        "push $" #def_func "\n"  /* Fill out prog_ctr with known value */ \
+        "push $0 \n" /* Leave space for prog_ctr */ \

[hidehiko, 2014/09/16 16:34:00]

nit: how about align the spacing with the above line?

[Junichi Uekawa, 2014/09/16 20:57:31]

Done.

         "push %edi\n" \
         "push %esi\n" \
         "push %ebp\n" \
@@ -252,6 +252,12 @@ extern const uint8_t kX86FlagBits[5];
         /* Save flags. */ \
         SAVE_X86_FLAGS_INTO_REG("%eax") \
         "movl %eax, 0x24(%esp)\n" \
+        /* Obtain def_func from GOT to fill prog_ctr with known value. */ \
+        "call 0f\n" \
+        "0: popl %eax\n" \
+        "1: addl $_GLOBAL_OFFSET_TABLE_ + (1b - 0b), %eax\n" \
+        "movl " #def_func "@GOT(%eax), %eax\n" \
+        "movl %eax, 0x20(%esp)\n" \
         /* Adjust saved %esp value to account for preceding pushes. */ \
         "addl $5 * 4, 0x10(%esp)\n" \
         /* Save argument to callee_func() temporarily. */ \