https://github.com/frasercrmck closed
https://github.com/llvm/llvm-project/pull/86965
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
frasercrmck wrote:
> The build here seems to be trying to define clc as a language, which then
> results in needing to rely on language support magic like this. I think it
> would be better if this did what rocm-device-libs does and treat these as
> custom targets. I don't think it's
https://github.com/arsenm approved this pull request.
In the context of what the build is already doing, this should be fine
https://github.com/llvm/llvm-project/pull/86965
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
arsenm wrote:
The build here seems to be trying to define clc as a language, which then
results in needing to rely on language support magic like this. I think it
would be better if this did what rocm-device-libs does and treat these as
custom targets. I don't think it's particularly helpful
https://github.com/frasercrmck created
https://github.com/llvm/llvm-project/pull/86965
This commit fixes the problem of missing build dependencies between libclc
source files and their various includes (namely headers and .inc files).
We would like to do this with compiler-generated