rapidsai/cuml
[DEBT] rename .h|.hpp|.cuh|.cu|.cpp files properly in our cuml C++ source base
开放
#1,675 创建于 2020年2月13日
Build or DepCUDA / C++Tech Debtgood first issueproposal
仓库指标
- 星标
- (5,223 个星标)
- PR 合并指标
- (PR 指标待抓取)
描述
Currently, they are just all over the place! I propose the following rule-of-thumb:
- If a header file contains cuda kernels and/or device methods, then it should be a
.cuh. - Else, it should be a
.hpp(if the current header file is already named as a.h, feel free to leave it as it is, in order to minimize the amount of ripple changes this will cause). - If a header file is used to declare our C interface, it should be a
.h - If the header file includes a
.cuh, then it should be a.cuhas well - If a source includes a
.cuh, then it should be a.cu, else it should be a.cpp - All source files defining our C interface should be
.c
This will certainly help us in reducing compilation time as well (if we end up cleaning some of the .cu files and renaming them to .cpp).