This change skips find_package(RV) if the RV target was already imported as part of LLVM and makes RV a public dependency of thorin to allow for transitive dependencies. It also changes the thorin CMake import to require LLVM and RV if thorin was compiled with LLVM+RV support. Having Thorin_HAS_LLVM_SUPPORT/Thorin_HAS_RV_SUPPORT report True without LLVM/RV actually being there does not seem very useful, and having the respective imports fail quietly makes debugging build issues where the thorin build finds the packages but the import of thorin into another component doesn't find them pretty frustrating.
This change skips
find_package(RV)if the RV target was already imported as part of LLVM and makes RV a public dependency of thorin to allow for transitive dependencies. It also changes the thorin CMake import to require LLVM and RV if thorin was compiled with LLVM+RV support. HavingThorin_HAS_LLVM_SUPPORT/Thorin_HAS_RV_SUPPORTreportTruewithout LLVM/RV actually being there does not seem very useful, and having the respective imports fail quietly makes debugging build issues where the thorin build finds the packages but the import of thorin into another component doesn't find them pretty frustrating.