malaperle added a comment. In https://reviews.llvm.org/D44251#1031370, @ilya-biryukov wrote:
> LGTM. > I think this was part of initial changed and I asked to make it this way, > but I don't remember why. I think it was just to have a single place that returns. I think it's OK to early return in this case, it's a good optimization. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D44251 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits