[libcxx-commits] [PATCH] D99398: [libcxx] reworks invocable and regular_invocable tests

[Martin Storsjö via Phabricator via libcxx-commits]

mstorsjo added inline comments.


================
Comment at: libcxx/test/std/concepts/concepts.callable/concept.invocable/invocable.compile.pass.cpp:232
+static_assert(std::invocable<multiple_overloads&, int>);
+static_assert(!std::invocable<multiple_overloads&, long>);
+static_assert(std::invocable<multiple_overloads&, double>);
----------------
This static assert (and the other one regarding `long` below on line 237, and two asserts in the other file) fails when building for Windows. On Windows (both 32 and 64 bit), long is the same size as int. Not familiar with why this passes on 32 bit linux then (where long also is the same size as int) but not on Windows - apparently there are differences in how distinct the two types are considered, or something else?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99398