[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