zero9178 accepted this revision.
zero9178 added a comment.

Thanks a lot for the patch :) Wanted this bit of code to be removed for quite 
some time.

> Since D103773 <https://reviews.llvm.org/D103773>, clang-cl's default mode 
> completely disables C++ operator
> names. That means that query.h compiles out of the box without this hack
> in the pre-processor. For users who are using /permissive-, they must
> set the QUERY_H_RESTRICTION_PERMISSIVE macro to rename this problematic
> or field.

Just a heads up, the patch added `/permissive` and `/permissive-`, but made 
neither the default as to not be a breaking change. That means that clang-cl 
still parses operator keywords and also does delayed template parsing, as it 
did before the patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108720

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

Reply via email to