================
@@ -4224,15 +4224,18 @@ class DecompositionDecl final
     : public VarDecl,
       private llvm::TrailingObjects<DecompositionDecl, BindingDecl *> {
   /// The number of BindingDecl*s following this object.
-  unsigned NumBindings;
+  unsigned NumBindings : 31;
+
+  LLVM_PREFERRED_TYPE(bool)
+  unsigned IsDecisionVariable : 1;
----------------
cor3ntin wrote:

I am questioning if we really need that.
We know from context if it is or not (ie it's not really a property of the 
declaration)

- In a _condition_ a DecompositionDecl is always a decision variable
- Outside, it never is.

Maybe
`Compiler<Emitter>::visitDeclStmt`, `CodeGenFunction::EmitDecl`, `EvaluateDecl` 
can be modified
to take a bool that indicates whether we are in the context of a _condition_.

Is that an approach you considered?





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

Reply via email to