[PATCH] D28217: [libc++] Overallocation of am_pm array in locale.cpp

[Marshall Clow via Phabricator via cfe-commits]

mclow.lists added a comment.

This looks fine to me.  
All the other places that use this (or related functionality) have a rank of 2.

Interestingly enough, this has been there since "the beginning of time" (the initial import of libc++ into the LLVM subversion repo)


https://reviews.llvm.org/D28217