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

[Kadir Cetinkaya via Phabricator via llvm-commits]

kadircet added a comment.

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.

If so, why do we need that exactly? What's wrong with specifying the config/response file as:

  --target=sample
  -isystem
   @include
  -L
   @lib
  -T 
  @ldscripts/link.ld

instead of having them on the flag and the value on the same line? are there any "real" cases in which you need a `foo=@something` today and that command line option doesn't have a form without the `=` in between?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115604