[PATCH] D44036: OpenBSD UBsan support / common part 3

David CARLIER via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 2 14:23:58 PST 2018


devnexen added a comment.

In https://reviews.llvm.org/D44036#1026030, @vitalybuka wrote:

> Title of the patch is not useful, but it hard to make it as this patch is still too large to my taste.
>  E.g. There is no reason to have ReadProcMaps and GetTls changes in a single patch.
>
> Ideally:
>  One patch for trivial "SANITIZER_OPENBSD ||"
>  For the rest. One patch per fixed function.


Ok but the GetTid change relies a bit on my other patch


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D44036





More information about the llvm-commits mailing list