JonChesterfield added a comment.

Testing manually looks good, provided there's no command line argument 
involved. -rpath even composes sanely, so doing this plus passing -Wl,rpath= 
results in multiple embedded rpaths. At a loss as to why changing Options.td is 
working so poorly, though it may be relevant that ninja thinks there's no work 
to do when the file changes. Trying things like:

def fno_openmp_add_rpath: Flag<["-"], "fno-openmp-add-rpath">, 
Flags<[NoArgumentUnused]>,

  HelpText<"Do not add -rpath with architecture-specific resource directory to 
the linker flags">;

which is a direct copy of fno_rtlib_add_path, didn't even bother changing the 
help text, and it's still missing from the clang executable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118493

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

Reply via email to