【第三方库离线编译】add submodule pocketfft and xbyak #54344
Conversation
|
你的PR提交成功，感谢你对开源项目的贡献! |
RedContritio
force-pushed
the
offline_cpx
branch
3 times, most recently
from
02d7afc
to
7d74285
Compare
RedContritio
force-pushed
the
offline_cpx
branch
3 times, most recently
from
f5455b1
to
426412c
Compare
|
related issue https://github.com/PaddlePaddle/Paddle/issues/54350， hi @risemeup1 @RedContritio, 我们能先保持cryptopp不动， 讨论下用 https://github.com/abdes/cryptopp-cmake 这个库替代的可行性吗？ 貌似 https://github.com/noloader/cryptopp-cmake 已经 archive 不再维护， 原作者也在 readme 中推荐使用 https://github.com/abdes/cryptopp-cmake |
ok，好的，那就暂时不动cryptopp |
好的了解，我看看 |
RedContritio
force-pushed
the
offline_cpx
branch
2 times, most recently
from
cfb70ab
to
03a3ae9
Compare
RedContritio
changed the title
|
@risemeup1 待 review |
好的，明天给你合入 |
| COMMAND ${CMAKE_COMMAND} -E remove_directory ${POCKETFFT_SOURCE_DIR} | ||
| COMMAND ${CMAKE_COMMAND} -E make_directory ${POCKETFFT_SOURCE_DIR} | ||
| COMMAND ${CMAKE_COMMAND} -E copy_directory ${SOURCE_DIR} | ||
| ${POCKETFFT_SOURCE_DIR} | ||
| INSTALL_COMMAND "" |
There was a problem hiding this comment.
因为现存代码里有对 extern_pocketfft 目录的引用：
Paddle/paddle/phi/kernels/funcs/fft.cc
Lines 21 to 27 in c15e53d
为了保留外部代码的一致性，所以需要复制一份到 ${THIRD_PARTY}/pocketfft/src/extern_pocketfft，并且将 ${THIRD_PARTY}/pocketfft/src 作为 include_directories
There was a problem hiding this comment.
如果不复制的话，就需要将 ${PADDLE_SOURCE_DIR}/third_party 作为 include_directories 了，我觉得暴露这个目录不太优雅。
一个可选的思路的将保留 extern_pocketfft 这个目录，但是仅复制需要的 "extern_pocketfft/pocketfft_hdronly.h"
There was a problem hiding this comment.
因为现存代码里有对
extern_pocketfft目录的引用：Paddle/paddle/phi/kernels/funcs/fft.cc
Lines 21 to 27 in c15e53d
为了保留外部代码的一致性，所以需要复制一份到
${THIRD_PARTY}/pocketfft/src/extern_pocketfft，并且将${THIRD_PARTY}/pocketfft/src作为include_directories
#include "extern_pocketfft/pocketfft_hdronly.h" 改为 #include "pocketfft_hdronly.h" 应该也可以吧，然后include_directories(SOURCE_DIR)?
There was a problem hiding this comment.
如果不复制的话，就需要将
${PADDLE_SOURCE_DIR}/third_party作为include_directories了，我觉得暴露这个目录不太优雅。一个可选的思路的将保留 extern_pocketfft 这个目录，但是仅复制需要的
"extern_pocketfft/pocketfft_hdronly.h"
复制也是一种方法，但是系统本身有一份再进行复制是不是冗余操作？这样暴露不太优雅？指的是？
There was a problem hiding this comment.
如果不复制的话，就需要将
${PADDLE_SOURCE_DIR}/third_party作为include_directories了，我觉得暴露这个目录不太优雅。
一个可选的思路的将保留 extern_pocketfft 这个目录，但是仅复制需要的"extern_pocketfft/pocketfft_hdronly.h"复制也是一种方法，但是系统本身有一份再进行复制是不是冗余操作？这样暴露不太优雅？指的是？
是这样的，参考 pybind 的引用，我是倾向于 #include "<third_part_name>/<header_filename>" 这种形式。
之所以之前没有改成 #include "pocketfft_hdronly.h" ，是因为感觉不能直观看出是第三方库。
pybind 那边的目录结构是 pybind/include/pybind11/xxx.h，所以能够直接将 ${SOURCE_DIR}/include 作为 include 目录；
不优雅指的是，不同于 pybind， pocketfft 目录是展开的，如果想保留 <库名>/<头文件名> 的方式，就可能导致 ${PADDLE_SOURCE_DIR}/third_party 目录暴露，担心后续可能会出现 include 冲突的情况，"${PADDLE_SOURCE_DIR}/third_party/xxx" 和其他位置的 xxx 冲突；
如果修改 INCLUDE_DIR 到 SOURCE_DIR，会导致在 #include 的时候看不到库名。
出于这个考虑，所以直接复制了整个目录到原来的 extern_pocketfft，来保证外部引用的一致。
There was a problem hiding this comment.
可以，那就复制吧，的确，肉眼第一时间看出来某个第三方库的头文件是比较好的。good job
PR types
Others
PR changes
Others
Description
将 pocketfft 和 xbyak 作为子模块引入，以支持离线编译。