[libcxx-commits] [PATCH] D122276: [libc++] Implement P0154R1 (Hardware inference size)

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added a comment.

Do you know whether Clang intends to implement this and if so whether they will use the same approach?
Since GCC 12 hasn't been released yet there's no CI testing done. So I would prefer to wait with this patch until we have GCC 12 in the CI.



================
Comment at: libcxx/docs/Status/Cxx17Papers.csv:38
 "`P0025R0 <https://wg21.link/P0025R0>`__","LWG","An algorithm to ""clamp"" a value between a pair of boundary values","Jacksonville","|Complete|","3.9"
-"`P0154R1 <https://wg21.link/P0154R1>`__","LWG","constexpr std::hardware_{constructive,destructive}_interference_size","Jacksonville","",""
+"`P0154R1 <https://wg21.link/P0154R1>`__","LWG","constexpr std::hardware_{constructive,destructive}_interference_size","Jacksonville","|Complete|","15.0"
 "`P0030R1 <https://wg21.link/P0030R1>`__","LWG","Proposal to Introduce a 3-Argument Overload to std::hypot","Jacksonville","|Complete|","3.9"
----------------
I don't think we can claim we're implemented this. ATM it's not implemented in Clang.
At the very least we need to add a note explaining the compiler dependency.
(Requiring using the GCC compiler means it doesn't work for all users, for example AppleClang users.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122276