[PATCH] D49867: [test] Use printf instead of C++ iostream, NFC.

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 26 11:14:33 PDT 2018


vsk added a comment.

In https://reviews.llvm.org/D49867#1177087, @Hahnfeld wrote:

> In https://reviews.llvm.org/D49867#1177081, @vsk wrote:
>
> > Thanks, LGTM. So, am I understanding correctly that there are msan interceptors for printf, but not iostream?
>
>
> I'm not sure that there are interceptors for `printf`. I think the problem was that instrumented and inlined code from headers got mixed with uninstrumented code from the shared library.


Oh, whereas none of the printf logic is inline, so either all of it is instrumented or not instrumented. Sure, makes sense.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D49867





More information about the llvm-commits mailing list