================
@@ -1183,7 +1181,7 @@ class Sema final : public SemaBase {
   std::optional<std::unique_ptr<DarwinSDKInfo>> CachedDarwinSDKInfo;
   bool WarnedDarwinSDKInfoMissing = false;
 
-  bool WarnedStackExhausted = false;
+  SingleWarningStackAwareExecutor StackAwareExecutor;
----------------
bricknerb wrote:

Yes, I considered that and it seems somewhat to add some complexity.
Assuming the same code might hit this in both Sema and CodeGen, I don't see a 
reason the user should see this warning twice.
If Sema and CodeGen would typically be hitting different use cases, it might 
make sense, but I don't think that's the case.
It's anyways beyond the scope of this change, and we could consider it as a 
future improvement, though I doubt it should be high priority.

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

Reply via email to