[PATCH] D118493: [WIP]Set rpath on openmp executables

[Jon Chesterfield via Phabricator via cfe-commits]

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