win: support x64 reading x86 (wow64)

snapshot/win/process_reader_win.h

Index: snapshot/win/process_reader_win.h
diff --git a/snapshot/win/process_reader_win.h b/snapshot/win/process_reader_win.h
index f23d0d5375d496e94357b543747ff9abb753ed27..73ad1c610078de46d090ddef07c99b5fb75a324a 100644
--- a/snapshot/win/process_reader_win.h
+++ b/snapshot/win/process_reader_win.h
@@ -20,6 +20,7 @@
 
 #include <vector>
 
+#include "build/build_config.h"
 #include "util/misc/initialization_state_dcheck.h"
 #include "util/win/address_types.h"
 #include "util/win/process_info.h"
@@ -43,7 +44,12 @@ class ProcessReaderWin {
     Thread();
     ~Thread() {}
 
-    CONTEXT context;
+    union {
+      CONTEXT native;
+#if defined(ARCH_CPU_64_BITS)
+      WOW64_CONTEXT wow64;
+#endif;
+    } context;
     uint64_t id;
     WinVMAddress teb;
     WinVMAddress stack_region_address;
@@ -108,7 +114,7 @@ class ProcessReaderWin {
 
  private:
   template <class Traits>
-  void ReadThreadData();
+  void ReadThreadData(bool is_64_reading_32);

[Mark Mentovai, 2015/09/18 22:10:50]

Yeah, this is better. My problem with is_wow64 was that it’d be false with a
32-bit handler on a 64-bit OS, but it’s not correct to say that the 32-bit
crashy app is not WoW64.

[scottmg, 2015/09/18 22:25:00]

Gotcha.

 
   HANDLE process_;
   ProcessInfo process_info_;