[PATCH] D35181: Defer addition of keywords to identifier table when loading AST

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM aside from a few small nits.



================
Comment at: include/clang/Basic/IdentifierTable.h:471
+  /// \brief Create the identifier table.
+  IdentifierTable(IdentifierInfoLookup *externalLookup = nullptr);
+
----------------
`ExternalLookup` to match coding conventions. Also, can you mark this constructor as explicit? Might as well change the other constructor as well since it's being reformatted.


Repository:
  rC Clang

https://reviews.llvm.org/D35181