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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to