[PATCH] D17264: [Compiler-rt][MSan] Fix msan_print_shadow3.cc test for Big Endian archs

Mohit Bhakkad via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 24 00:43:59 PST 2016


mohit.bhakkad updated this revision to Diff 48892.
mohit.bhakkad added a comment.

I have removed endian based conditions for shadow printing in http://reviews.llvm.org/D17563.
This revision uses a char array to make this test compatible for both endians.


Repository:
  rL LLVM

http://reviews.llvm.org/D17264

Files:
  test/msan/msan_print_shadow3.cc

Index: test/msan/msan_print_shadow3.cc
===================================================================
--- test/msan/msan_print_shadow3.cc
+++ test/msan/msan_print_shadow3.cc
@@ -6,7 +6,7 @@
 
 int main(void) {
   unsigned long long x = 0; // For 8-byte alignment.
-  uint32_t x_s = 0x12345678U;
+  char x_s[4] = {0x87, 0x65, 0x43, 0x21};
   __msan_partial_poison(&x, &x_s, sizeof(x_s));
   __msan_print_shadow(&x, sizeof(x_s));
   return 0;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17264.48892.patch
Type: text/x-patch
Size: 446 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160224/79e5ce92/attachment.bin>


More information about the llvm-commits mailing list