[PATCH] D115604: [Support] Expand `<@>` as the base directory in response files.

Jack Andersen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 13 06:08:56 PST 2021


jackoalan added a comment.

In D115604#3188392 <https://reviews.llvm.org/D115604#3188392>, @kadircet wrote:

> IIUC, the new behavior being introduced by this patch is not the ability of having a way to refer to other files in a config/response file relative way, but rather extending that ability from only the options that start with `@` to options that contain `@` as a sub-string in any place.

`@` actually behaves as an "include" operation (i.e. the contents of the relative file are tokenized into more response file args). It will fail if a directory is passed because it is only used by `FileSystem::getBufferForFile`.

The `<@>` operation I am proposing does not perform any file load and simply expands to the base path as the function already knows it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115604



More information about the llvm-commits mailing list