EricWF added inline comments.

================
Comment at: src/experimental/filesystem/operations.cpp:22-24
+#if defined(__APPLE__)
+#include <Availability.h>
+#endif
----------------
dexonsmith wrote:
> I only just noticed you were including Availability.h.  That shouldn't be 
> necessary, since the macros should be defined by the compiler.
__MAC_10_13 et al are defined in `Availability.h`, and `AvailabilityInternal.h` 
seems to do the `__ENV` dance described above. Are you sure it's not needed?


https://reviews.llvm.org/D34249



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to