[PATCH] D83499: [MSAN runtime] Add poison_stack function that also updates origin

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 9 17:55:57 PDT 2020


vitalybuka accepted this revision.
vitalybuka added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/test/Instrumentation/MemorySanitizer/alloca.ll:9
+
+; RUN: opt < %s -msan-check-access-address=0 -msan-poison-stack-with-call=1    \
+; RUN: -msan-track-origins=1 -S -passes=msan 2>&1 | FileCheck %s "--check-prefixes=CHECK,CALL-ORIGIN"
----------------
flag order is inconsistent
also line breaks with \ is unnecessary 
long lines should be accepted by clang-format -style file in tests


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83499





More information about the llvm-commits mailing list