[PATCH] D34440: [Clang] Expand response files before loading compilation database

[Anton Korobeynikov via Phabricator via cfe-commits]

asl added a comment.

In https://reviews.llvm.org/D34440#809525, @alexfh wrote:

> In https://reviews.llvm.org/D34440#808156, @vladimir.plyashkun wrote:
>
> > **To discuss:**
> >  ...
> >  By this moment, we unable to use //CompilationDatabase.json// from //CLion //side which is widely used in //Clang-Tidy// and in other common tools.
>
>
> It would be interesting to learn more about the reasons why you can't use JSON compilation database. In case you don't want to clutter the project's directory, you can place the compile_commands.json file elsewhere (in a temporary directory, for example) and point clang tools to this directory using the `-p` command line flag.


Many build systems normally generate response files on-fly in some circumstances (e.g. if command line is longer than some platform-imposed limit). So IMO response files should be a perfect citizen here.


Repository:
  rL LLVM

https://reviews.llvm.org/D34440