[libcxx-commits] [PATCH] D141334: [libc++][test] Silence allocator conversion warnings

[Casey Carter via Phabricator via libcxx-commits]

CaseyCarter added a comment.

In D141334#4040566 <https://reviews.llvm.org/D141334#4040566>, @ldionne wrote:

> CI issues seem unrelated. Just curious -- how did you catch that? Is there anything we can/should add to our implementation?

As you may be aware, MSVC doesn't have the "system header" notion that GCC/Clang do. Consequently we run testing with TONS of warnings enabled to ensure that our header code only emits warnings when the user has asked us to do something questionable. This is a pretty good example: `Allocator`'s size type is `size_t`, so using this allocator with Standard Library containers results in lots of narrowing conversions from size_t to int. If the user asks for narrowing conversion warnings, they'll get them. The consequence is we have to police our tests pretty carefully to avoid warnings so we only see warnings from bad product code.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141334/new/

https://reviews.llvm.org/D141334