[all-commits] [llvm/llvm-project] db0ac3: [libc++] Fix warning C4244 in std/numerics/rand/ra...

Igor Zhukov via All-commits all-commits at lists.llvm.org
Tue Aug 2 17:46:53 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: db0ac307c9df26d26a629552aec0a78f1b492dfd
      https://github.com/llvm/llvm-project/commit/db0ac307c9df26d26a629552aec0a78f1b492dfd
  Author: Igor Zhukov <fsb4000 at yandex.ru>
  Date:   2022-08-02 (Tue, 02 Aug 2022)

  Changed paths:
    M libcxx/test/std/numerics/rand/rand.dist/rand.dist.samp/rand.dist.samp.discrete/eval.pass.cpp

  Log Message:
  -----------
  [libc++] Fix warning C4244 in std/numerics/rand/rand.dist/rand.dist.samp/rand.dist.samp.discrete/eval.pass.cpp

frederick-vs-ja noticed that https://github.com/microsoft/STL/pull/2976#issuecomment-1201926893
while we are working on updating LLVM submodule for MS STL:

    [...]\std\numerics\rand\rand.dist\rand.dist.samp\rand.dist.samp.discrete\eval.pass.cpp(33): error C2220: the following warning is treated as an error
    [...]\std\numerics\rand\rand.dist\rand.dist.samp\rand.dist.samp.discrete\eval.pass.cpp(287): note: see reference to function template instantiation 'void tests<__int64>(void)' being compiled
    [...]\std\numerics\rand\rand.dist\rand.dist.samp\rand.dist.samp.discrete\eval.pass.cpp(33): warning C4244: 'argument': conversion from '__int64' to 'const unsigned int', possible loss of data

Differential Revision: https://reviews.llvm.org/D130963


  Commit: 495519e5f8232d144ed26e9c18dbcbac6a5f25eb
      https://github.com/llvm/llvm-project/commit/495519e5f8232d144ed26e9c18dbcbac6a5f25eb
  Author: Igor Zhukov <fsb4000 at yandex.ru>
  Date:   2022-08-02 (Tue, 02 Aug 2022)

  Changed paths:
    M libcxx/test/std/algorithms/ranges_result_alias_declarations.compile.pass.cpp

  Log Message:
  -----------
  [libc++][NFC] Don't rely on `<algorithm>` transitively including `<memory>` in tests

Found by @cpplearner (https://github.com/microsoft/STL/pull/2976#discussion_r935440806)

Differential Revision: https://reviews.llvm.org/D130997


Compare: https://github.com/llvm/llvm-project/compare/d434e40f398e...495519e5f823


More information about the All-commits mailing list