[PATCH] D17380: [libcxx] Split locale management out of ibm/xlocale.h. NFCI

Ben Craig via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 18 07:39:29 PST 2016


bcraig created this revision.
bcraig added reviewers: howard.hinnant, mclow.lists.
bcraig added a subscriber: cfe-commits.

This is one part of many of a locale refactor.  See http://reviews.llvm.org/D17146 for an idea of where this is going.

For the locale refactor, the locale management functions (newlocale, freelocale, uselocale) are needed in a separate header from the various _l functions.  This is because some platforms implement the _l functions in terms of a locale switcher RAII helper, and the locale switcher RAII helper needs the locale management functions.  This patch helps pave the way by getting all the functions in the right files, so that later diffs aren't completely horrible.

Unfortunately, I have no access to an AIX machine to build with, so this change has been made blind.  Also, the original author (Xing Xue) does not appear to have a Phabricator account.

http://reviews.llvm.org/D17380

Files:
  include/support/ibm/locale_mgmt_aix.h
  include/support/ibm/xlocale.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17380.48314.patch
Type: text/x-patch
Size: 4974 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160218/044b76f8/attachment.bin>


More information about the cfe-commits mailing list