[PATCH] D54739: [Sanitizer] intercept setvbuf on other platforms where it is supported

[David CARLIER via Phabricator via llvm-commits]

devnexen added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_platform_interceptors.h:519
 #define SANITIZER_INTERCEPT_NETENT SI_NETBSD
-#define SANITIZER_INTERCEPT_SETVBUF SI_NETBSD
+#define SANITIZER_INTERCEPT_SETVBUF (SI_NETBSD || SI_FREEBSD || SI_LINUX || SI_MAC)
 #define SANITIZER_INTERCEPT_GETMNTINFO SI_NETBSD
----------------
vitalybuka wrote:
> we don't want to enable this without testing
> so either remove new plaform from here or remove it from Unsupported
> Mac and linux is safe to enable. we have bots and should quckly notice issues
> I still suspect that unpoisonfile is needed.
>   
Yes it is a bit of "WIP" differential for now, did not get expect to be accepted as is. I m testing a Linux build this morning.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D54739