[PATCH] Newlib names ELAST differently than linux

Jon Roelofs jonathan at codesourcery.com
Tue Aug 26 15:51:46 PDT 2014


That sounds like a good idea. Maybe it's better to have it be:

#if defined(ELAST)
#define LIBCPP_ELAST ELAST
#elif defined(__linux__)
#define LIBCPP_ELAST 4095
#elif defined(_NEWLIB_VERSION) && defined(__ELASTERROR)
#define LIBCPP_ELAST __ELASTERROR
#else
// Error here so that the person doing the libcxx port has an easier time:
#error This platform's ELAST hasn't been ported yet
#endif
#endif

... as I don't think it's appropriate for __config to be defining ELAST so globally.

http://reviews.llvm.org/D5079






More information about the cfe-commits mailing list