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

Kostya Serebryany via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 26 10:58:37 PDT 2017


kcc added a comment.

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


Repository:
  rL LLVM

https://reviews.llvm.org/D35554





More information about the llvm-commits mailing list