[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