rapidsai/cuml

[DEBT] rename .h|.hpp|.cuh|.cu|.cpp files properly in our cuml C++ source base

オープン

#1,675 opened on 2020/02/13

 (44 件のコメント) (0 件のリアクション) (1 人の担当者)Python (644 件のフォーク)auto 404
Build or DepCUDA / C++Tech Debtgood first issueproposal

Repository metrics

Stars
 (5,223 個のスター)
PR merge metrics
 (PR metrics pending)

説明

Currently, they are just all over the place! I propose the following rule-of-thumb:

  1. If a header file contains cuda kernels and/or device methods, then it should be a .cuh.
  2. 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).
  3. If a header file is used to declare our C interface, it should be a .h
  4. If the header file includes a .cuh, then it should be a .cuh as well
  5. If a source includes a .cuh, then it should be a .cu, else it should be a .cpp
  6. 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).

コントリビューターガイド