[PATCH] D106206: Lazy initialized the NotUnderValgrind flag

Lang Hames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 16 21:46:06 PDT 2021


lhames added a comment.

This code predates my involvement in the JIT by half a decade or more. ;)

It's definitely reachable, but I'm not sure whether anyone actually relies on it any more. My first impression is that running Valgrind on JIT'd code would be pretty rare these days. Maybe we just remove it after we branch for LLVM 13 in a week or so?


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