[libcxx-commits] [PATCH] D96786: <ciso646> should provide defines for alternative operator representations

[Yuriy Chernyshov via Phabricator via libcxx-commits]

georgthegreat added a comment.

> MSVC only recognizes the alternative tokens in strict (`/permissive-`) mode to avoid breaking legacy code that uses those lexemes as identifiers. I have no opinion on whether or not libc++ should accept this change, but I strongly recommend that C++ code that wants portable conformant behavior when compiling with MSVC should use `/permissive-`, which will finally be on-by-default in C++20 mode.

It looks like `/permissive-` did the trick. Thanks for pointing this option out. I will try to integrate this option into build process.

Current change is definitely incorrect, including this header should break compilation if the code is compiled against modern C++ standard.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96786