royjacobson marked 2 inline comments as done. royjacobson added inline comments.
================ Comment at: clang/lib/Sema/SemaOverload.cpp:9829 + bool CanCompareConstraints = false; + if (Cand1.Function && Cand2.Function && Cand1.Function->hasPrototype() && + Cand2.Function->hasPrototype()) { ---------------- erichkeane wrote: > royjacobson wrote: > > erichkeane wrote: > > > Since the point of this is to just calculate the CanCompareConstraints, I > > > think it should be a separate function called below where it is used. > > Do you mean as in a separate `Sema` function? Or a local lambda? > Just a static function is fine, basically its a whole lot of work happening > in this function for a single result, so I'm suggesting splitting off the > calculation for `CanCompareConstraints` into its own function, so you get: > > `bool CanCompareConstrants = new_function(...);` Done Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123182/new/ https://reviews.llvm.org/D123182 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits