[PATCH] D41537: Optionally add code completion results for arrow instead of dot

[Ivan Donchevskii via Phabricator via cfe-commits]

yvvan marked 4 inline comments as done.
yvvan added inline comments.


================
Comment at: lib/Sema/SemaCodeComplete.cpp:4148
+
+  CompletionSucceded |= DoCompletion(Base, IsArrow, None);
+
----------------
ilya-biryukov wrote:
> NIT: maybe swap the two cases to do the non-fixit ones first? It is the most common case, so it should probably go first.
The actual completion should be the last because of the "Completion context". I've put them in this order intentionally. I will add comment about that.


https://reviews.llvm.org/D41537