Mordante marked 3 inline comments as done. Mordante added inline comments.
================ Comment at: clang/lib/Sema/SemaChecking.cpp:12349 // This is not the right CC for (e.g.) a variable initialization. - AnalyzeImplicitConversions(*this, E, CC); + analyzeImplicitConversionsWithSufficientStackSpace(*this, E, CC); } ---------------- xbolva00 wrote: > xbolva00 wrote: > > Please do not change function names. > Ah, you had to change it. > > Can this be solved maybe via new bool argument for > AnalyzeImplicitConversions(.., arg)? If arg is false, we could just call > S.runWithSufficientStackSpace(AnalyzeImplicitConversions(...., true)), > otherwise continue in normal flow. I like this idea better than my original approach. Thanks for the suggestion. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69479/new/ https://reviews.llvm.org/D69479 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
