================
@@ -853,6 +853,14 @@ class CIRGenFunction : public CIRGenTypeCache {
                      FunctionArgList args, clang::SourceLocation loc,
                      clang::SourceLocation startLoc);
 
+  /// returns true if aggregate type has a volatile member.
+  /// TODO(cir): this could be a common AST helper between LLVM / CIR.
+  bool hasVolatileMember(QualType t) {
----------------
andykaylor wrote:

I'm not sure that really makes sense. The logical place to put it would be in 
`QualType`, but it is only meaningful if this is a record type. In theory we 
could move the `getAsRecordDecl` check inside `QualType`, but that doesn't feel 
right. I'm inclined to just remove the comment about sharing.

https://github.com/llvm/llvm-project/pull/163877
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to