cachemeifyoucan wrote:

> I noticed LLVM_CAS_PLUGIN_TEST_PATH is a bit different from how other test 
> plugins are found: it's an absolute path, and it recomputes the output dir 
> that unittests/CAS/CMakeLists.txt sets. Could this follow CGTestPlugin 
> instead, i.e. leave the plugin in the lib dir and use %llvmshlibdir / 
> %pluginext in lit.local.cfg? That would remove the new site-config variable.

Sure. It was actually like originally but it was moved due to it was only used 
by unit tests. Now we have a tool to load that plugin as well, I should move it 
back.

https://github.com/llvm/llvm-project/pull/227056
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to