[PATCH] D68075: Do not #error if no OS is #defined

David Zarzycki via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 26 07:38:28 PDT 2019


davezarzycki added a comment.

I think trying to work out of the box with zero tuning in common scenarios and minimal tuning with uncommon scenarios is the right approach. If it were up to me, I’d flip the test: If windows then set the Windows define, otherwise pthreads.h exists, assume pthreads. If somebody is compiling “freestanding” like myself, then a link error about missing pthreads is expected behavior.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68075/new/

https://reviews.llvm.org/D68075





More information about the llvm-commits mailing list