[PATCH] D35554: Add NetBSD support in sanitizer_platform_limits_posix.*

Kamil Rytarowski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 27 05:39:45 PDT 2017


krytarowski abandoned this revision.
krytarowski added a comment.

In https://reviews.llvm.org/D35554#853339, @kcc wrote:

> This patch has too many #ifdefs. 
>  #ifdefs cause maintenance nightmare. 
>  Instead of #ifdefs, can you move the code to separate *netbsd*.[h,cpp] files?


I've submitted it as a new diff: https://reviews.llvm.org/D37193.


Repository:
  rL LLVM

https://reviews.llvm.org/D35554





More information about the llvm-commits mailing list