[PATCH] D44251: [clangd] Early return for #include goto definition.
Marc-Andre Laperle via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 8 07:36:12 PST 2018
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
More information about the cfe-commits
mailing list