diff options
author | Benoit Jacob <jacob.benoit.1@gmail.com> | 2024-10-04 12:13:06 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-10-04 12:13:06 -0400 |
commit | dd04cf0c9d97a6d73c832ceb6cd060ec3589876e (patch) | |
tree | 7b8e54e97f84fc5b10517d8dbf3422ee9dd7192a /clang/lib/CodeGen/CodeGenFunction.cpp | |
parent | 87c799af31e2a82801f4dbda83125a0265d8c216 (diff) | |
download | llvm-dd04cf0c9d97a6d73c832ceb6cd060ec3589876e.zip llvm-dd04cf0c9d97a6d73c832ceb6cd060ec3589876e.tar.gz llvm-dd04cf0c9d97a6d73c832ceb6cd060ec3589876e.tar.bz2 |
MLIR BUILD.bazel: fold `BasicPtxBuilderInterface` into `NVVMDialect` (#111172)
While doing an integrate into downstream
https://github.com/iree-org/iree, I ran into a typical Bazel error with
`BasicPtxBuilderInterface.cpp` including `NVVMDialect.h` which was not
exposed as a header by a declared dependency. I tried fixing this the
straightforward way, by letting `:BasicPtxBuilderInterface` depend on
`:NVVMDialect` , but that caused another Bazel error: circular
dependency between these two targets, as `:NVVMDialect` was already
depending on `:BasicPtxBuilderInterface`. I tried breaking that circle
by dropping the latter dependency, but it was a real dependency in the
code, specifically in the TableGen-generated code. So in the end it
seems that these two targets just need to be fused, which this PR does.
Signed-off-by: Benoit Jacob <jacob.benoit.1@gmail.com>
Diffstat (limited to 'clang/lib/CodeGen/CodeGenFunction.cpp')
0 files changed, 0 insertions, 0 deletions