[PATCH] D95423: [clangd] Add std::size_t to StdSymbol mapping

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 26 04:20:06 PST 2021


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

This might annoy people occasionally, but so does the current behavior.
I guess inserting *nothing* might be better, we could have a list of symbols known to have no insertable header. But we don't have that today, and this is an improvement we can land before the 12 release.

stddef seems like the best choice among the headers that provide size_t.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95423/new/

https://reviews.llvm.org/D95423



More information about the cfe-commits mailing list