Skip to content

Conversation

@Humber-186
Copy link
Collaborator

批量发起Ventus相关的各个仓库的开发分支到main分支的PR
此次将在ventus-env main分支当前最新提交上运行OpenCL CTS测试
已经核实过ventus-env中当前子仓库的commit号即为本PR中包含的最新commit号

@wenhu1024
Copy link
Collaborator

这个修改是什么原因

@Humber-186
Copy link
Collaborator Author

ventus-driver和libclc中存在同名但不同内容的ventus.h头文件,ventus-drver编译安装后会将其自身的ventus.h安装到install/include/ventus.h,如果之后再编译libclc,会优先使用install/include/ventus.h,而不是llvm-project/libclc/riscv32/lib/ventus.h造成编译报错。

这个提交会反转include路径的优先级,优先使用libclc自身的ventus.h规避此问题。
也许更好的方案是修改CMakeLists,不要添加install/include/这个include路径

@wenhu1024
Copy link
Collaborator

ventus-driver和libclc中存在同名但不同内容的ventus.h头文件,ventus-drver编译安装后会将其自身的ventus.h安装到install/include/ventus.h,如果之后再编译libclc,会优先使用install/include/ventus.h,而不是llvm-project/libclc/riscv32/lib/ventus.h造成编译报错。

这里我用./build-ventus.sh安装,在llvm-project/install/include/下面没有找到ventus.h ?

@Humber-186
Copy link
Collaborator Author

请用这个仓库中的build-ventus.sh,会安装到ventus-env/install:
https://github.com/THU-DSP-LAB/ventus-env

会将ventus.h安装到install的driver变更还没merge到driver的main分支,也在PR中

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants