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

[Wasim Abbas via Phabricator via cfe-commits]

abbaswasim added a comment.

> FWIW, I did a search over github repos and this file extension is used, but I would hardly call it a common practice

Fair enough. what would be sufficient evidence?

I think in general whether its `.inl` or any other extension thats used to separate template implementations out of a header needs a solution. I have started using `.hh` as a workaround but it not being a complete translation unit is still a problem. And clangd also complains about recursive includes in the .hh file.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67025