[PATCH] D106206: Lazy initialized the NotUnderValgrind flag

Jameson Nash via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 19 10:57:20 PDT 2021


vtjnash added a comment.

There doesn't seem to be a reason to make the `VALGRIND_DISCARD_TRANSLATIONS` call conditional. The macro is already a specific sequence of bytes (e.g. `rol` instructions) that are a no-op if valgrind is not running the particular segment of code inside its own JIT


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106206



More information about the llvm-commits mailing list