h-vetinari added a comment.

In D133853#3797598 <https://reviews.llvm.org/D133853#3797598>, @RIscRIpt wrote:

> I am afraid it would take me some effort to implement semantics of 
> `[[msvc::no_unique_address]]`, so I'd like to focus only on 
> `[[msvc::constexpr]]` in current patch.

Just for context, `[[msvc::no_unique_address]]` is 100% 
<https://devblogs.microsoft.com/cppblog/msvc-cpp20-and-the-std-cpp20-switch/#c20-no_unique_address>
 like the standardized `[[no_unique_address]]`, but MSFT was... exceedingly 
cautious... about introducing this, as - even for an explicitly opt-in new 
feature - someone might have compiled code containing `[[no_unique_address]]` 
at a time when it was a no-op in their compiler (as an unknown attribute), and 
changing the semantics after the fact would have violated their very strict 
compatibility guarantees.

I assume the reasoning behind `[[msvc::constexpr]]` to be along the same lines.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133853

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to