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

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 13 02:42:56 PST 2021


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



More information about the cfe-commits mailing list