[PATCH] D67025: Add .inl as valid C++ header type

Nathan Ridge via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 20 10:37:15 PDT 2022


nridge added a comment.

In D67025#3665293 <https://reviews.llvm.org/D67025#3665293>, @aaron.ballman wrote:

> this is why we have the `-x` option so that users can specify a specific language mode to use. Is there a reason that option does not suffice for clangd?

One reason is that the most commonly used tools that generate `compile_commands.json` files, like CMake, do not create entries for header files. Users need to turn to ad-hoc post-processing or third-party tools like CompDB <https://github.com/Sarcasm/compdb> to add entries for header files, to even have a place to put the `-x c++header` option.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67025



More information about the cfe-commits mailing list