================
@@ -503,17 +503,16 @@ bool Sema::checkLiteralOperatorId(const CXXScopeSpec &SS,
     const IdentifierInfo *II = Name.Identifier;
     ReservedIdentifierStatus Status = II->isReserved(PP.getLangOpts());
     SourceLocation Loc = Name.getEndLoc();
-    if (!PP.getSourceManager().isInSystemHeader(Loc)) {
----------------
erichkeane wrote:

It isn't clear to me that this check was necessary at all?  So I've removed it. 
 Both diags happen now without the check, and unconditionally on each-other.  
They also share the `FixItHint`, which was done in a frightening-ish way.

https://github.com/llvm/llvm-project/pull/111027
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to