Anastasia added inline comments.

================
Comment at: lib/Sema/SemaDeclCXX.cpp:8229
+  if (FTI.hasMethodTypeCVRUQualifiers()) {
+    FTI.MethodQualifiers->forEachCVRUQualifier(
         [&](DeclSpec::TQ TypeQual, StringRef QualName, SourceLocation SL) {
----------------
rjmccall wrote:
> rjmccall wrote:
> > We want to catch `_Atomic`, too, so please just change this loop to ignore 
> > address-space qualifiers, using a flag to decide whether to call 
> > `setInvalidType`.
> If there aren't any qualifiers we're skipping, the flag isn't necessary.
We are skipping addr space currently. I use this flag to avoid setting 
declaration as invalid below if it's only qualified by an addr space.


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

https://reviews.llvm.org/D62156



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

Reply via email to