ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.
LGTM modulo the comment about variable name.
================
Comment at: lib/Sema/SemaLookup.cpp:3502
+ bool IncludeDependentBases,
+ bool Load) {
if (!Ctx)
----------------
Maybe rename the variable to `LoadExternal`?
The intention of `Load` is not immediately clear. Also a bit longer name seems
to be more in-line with the other code in this file.
Repository:
rC Clang
https://reviews.llvm.org/D41989
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
