[PATCH] D44049: OpenBSD UBsan support enabling SANITIZER_OPENBSD widely

[David CARLIER via Phabricator via llvm-commits]

devnexen added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_posix_libcdep.cc:125
+#if SANITIZER_OPENBSD
+#define RLIMIT_AS RLIMIT_DATA
+#endif
----------------
vitalybuka wrote:
> Maybe make it more generic and avoid new definition?
> 
> 
> ```
> #if defined(RLIMIT_AS)
>   int rlimit =  RLIMIT_AS;
> #else
>   int rlimit = RLIMIT_DATA;
> #endif
>   rlim_t as_size = getlim(rlimit);
> ```
I could but RLIMIT_AS is still used in the function below.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D44049