[llvm] [mlir] [mlir] Move InlinerInterface from Transforms to Interfaces. (PR #84878)

Mehdi Amini via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 13 11:07:58 PDT 2024


joker-eph wrote:

> > I don't know how to model this with Bazel: would you create a "header only target" just for the header inclusion and another one for the link?
> 
> You can do that, but then a user of your dialect needs to explicitly add the link target as a dependency.

No: they are really header-only dependency, you don't need the link target.

You need to link to register the interface implementation only IIRC. 

CMake is lose with the header dependency, but has the same rules as Bazel for the link dependency (when we build with `-DBUILD_SHARED_LIBS=ON`)

https://github.com/llvm/llvm-project/pull/84878


More information about the llvm-commits mailing list