================
@@ -873,30 +877,17 @@ class Sema final : public SemaBase {
   /// Warn when implicitly casting 0 to nullptr.
   void diagnoseZeroToNullptrConversion(CastKind Kind, const Expr *E);
 
-  // ----- function effects ---
+  /// All functions/lambdas/blocks which have bodies and which have a non-empty
+  /// FunctionEffectsRef to be verified.
+  SmallVector<const Decl *> DeclsWithEffectsToVerify;
+  /// The union of all effects present on DeclsWithEffectsToVerify. Conditions
+  /// are all null.
+  FunctionEffectKindSet AllEffectsToVerify;
----------------
Endilll wrote:

Yes, grouping implementations in a single `SemaFunctionEffects.cpp` is the 
right first step if you want to group your functions. Based on that, you can 
create a new section in `Sema.h`, and put all declarations and data members 
there.

I'd refrain from a Sema subcomponent at this stage. My understanding of the 
efforts in this PR is that they are neither a language nor a target, and we 
don't know yet how it should be handled.

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

Reply via email to