================ @@ -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