[Mlir-commits] [mlir] [mlir][spirv] Add definition for selected sample operations (PR #129558)

[Jordan Rupprecht]

rupprecht wrote:

Is this a circular dep? IR depends on Interfaces (SPIRVOps.h includes SPIRVImageInterfaces.h) but Interfaces also depends on IR (SPIRVImageInterfaces.td includes SPIRVBase.td)

However, maybe it's just that bazel globs too much together, e.g. maybe some of the base files should be in individual targets, instead of just throwing *.h/*.cpp into a single target.

https://llvm.org/docs/CodingStandards.html#library-layering is the overall LLVM ref, but I don't know if that applies to MLIR in this way.

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