[libcxx-commits] [PATCH] D63284: [libc++] Keep __regex_word in sync with ctype_base
Mikhail Maltsev via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Tue Aug 20 03:07:21 PDT 2019
miyuki marked an inline comment as done.
miyuki added inline comments.
Comment at: libcxx/trunk/include/__locale:412
+ static const mask __regex_word = static_cast<char_class_type>(_ISbit(15));
> atanasyan wrote:
> > This change breaks building on mips: https://bugs.llvm.org/show_bug.cgi?id=43011
> > Maybe this change fix the problem, but I'm not an expert in libc++:
> > ```
> > static const mask __regex_word = static_cast<mask>(_ISbit(15));
> > ```
> Personally, I would lose the `_ISbit` call as well; just say `1 << 15` or `0x8000`
> We don't use `_ISbit` for the other constants, and we don't use it anywhere else in libc++.
_ISbit is endian-dependent, see https://reviews.llvm.org/D17132
CHANGES SINCE LAST ACTION
More information about the libcxx-commits