[all-commits] [llvm/llvm-project] 4e1c99: [llvm][test][CGPluginTest] Keep plugin in shared l...

[Raul Tambre via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4e1c996674cc340f290b0a528e2038e76494d8d4
      https://github.com/llvm/llvm-project/commit/4e1c996674cc340f290b0a528e2038e76494d8d4
  Author: Raul Tambre <raul at tambre.ee>
  Date:   2025-09-19 (Fri, 19 Sep 2025)

  Changed paths:
    M llvm/test/Other/codegen-plugin-loading.ll
    M llvm/unittests/CodeGen/CGPluginTest/CMakeLists.txt
    M llvm/unittests/CodeGen/CGPluginTest/Plugin/CMakeLists.txt
    M llvm/unittests/CodeGen/CGPluginTest/PluginTest.cpp

  Log Message:
  -----------
  [llvm][test][CGPluginTest] Keep plugin in shared library directory

Scoping to the root build directory instead of using the path directly is awkward and the only such
occurrence in the test suite. It's also prone to breakage for downstreams that change the library
path.

But it's not even necessary: during build we have the appropriate RPATHs set so we can just depend
on the dynamic loader to find it. This extra logic is probably just copy-paste from PluginsTest.cpp.

Additionally:
* Removed TargetParser as a dependency because it doesn't seem to actually be used.
* Moved `add_dependencies()` to `DEPENDS` to better match the rest of LLVM.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications