[PATCH] D88501: [asan][test] Several Posix/unpoison-alternate-stack.cpp fixes

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 30 04:01:50 PDT 2020


vitalybuka accepted this revision.
vitalybuka added a comment.
This revision is now accepted and ready to land.

Please do not include entire description into git commit.
A couple of sentences should be enough.



================
Comment at: compiler-rt/test/asan/TestCases/Posix/unpoison-alternate-stack.cpp:89
 
-  struct sigaction Action = {
-      .sa_sigaction = signalHandler,
-      .sa_flags = SA_SIGINFO | SA_NODEFER | SA_ONSTACK,
-  };
+  struct sigaction Action;
+  Action.sa_sigaction = signalHandler;
----------------



================
Comment at: compiler-rt/test/asan/TestCases/Posix/unpoison-alternate-stack.cpp:156
 
-  stack_t const AltStack = {
-      .ss_sp = (char *)Mapping + DefaultStackSize,
-      .ss_flags = 0,
-      .ss_size = AltStackSize,
-  };
+  stack_t AltStack;
+  AltStack.ss_sp = (char *)Mapping + DefaultStackSize;
----------------
same 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88501/new/

https://reviews.llvm.org/D88501



More information about the llvm-commits mailing list