include/sora/fix_cuda_noinline_macro_error.h を追加する

shiguredo · Jan 19, 2024 · 82de38c · 82de38c
1 parent ff87a14
commit 82de38c
Show file tree

Hide file tree

Showing 6 changed files with 38 additions and 11 deletions.
diff --git a/include/sora/fix_cuda_noinline_macro_error.h b/include/sora/fix_cuda_noinline_macro_error.h
@@ -0,0 +1,22 @@
+/*
+## 背景
+
+WebRTC を M121 に更新した際に以下のビルド・エラーが発生した
+
+```
+error: use of undeclared identifier 'noinline'; did you mean 'inline'
+```
+
+WebRTC に含まれる libcxx のバージョンが更新されたことが原因だと思われる
+
+## 対応
+
+同様の問題を解消したと思われる LLVM の [PR](https://github.com/llvm/llvm-project-release-prs/pull/698) を調査したところ、 PR で追加されたファイルは存在するにも関わらず、問題が継続して発生していることがわかった
+(libcxx に bits/basic_string.h が含まれておらず、 cuda_wrappers 以下のファイルがインクルードされていないようだった)
+
+上記 PR を参考に、ファイルを直接修正したところエラーが解消したため、このヘッダー・ファイルをエラーが発生する箇所でインクルードすることにした
+オリジナルのパッチには push_macro や pop_macro が含まれているが、省いても問題が無かったため省略している
+
+*/
+
+#undef __noinline__
diff --git a/include/sora/hwenc_nvcodec/nvcodec_decoder_cuda.h b/include/sora/hwenc_nvcodec/nvcodec_decoder_cuda.h
@@ -3,9 +3,10 @@
 
 #include <memory>
 
-// https://github.com/llvm/llvm-project-release-prs/pull/698/files
-#undef __noinline__
+// clang-format off
+#include "sora/fix_cuda_noinline_macro_error.h"
 #include <string>
+// clang-format on
 
 #include "sora/cuda_context.h"
 

diff --git a/third_party/NvCodec/NvCodec/NvDecoder/NvDecoder.cpp b/third_party/NvCodec/NvCodec/NvDecoder/NvDecoder.cpp
@@ -13,9 +13,10 @@
 #include <chrono>
 #include <cmath>
 
-// https://github.com/llvm/llvm-project-release-prs/pull/698/files
-#undef __noinline__
+// clang-format off
+#include "sora/fix_cuda_noinline_macro_error.h"
 #include <iostream>
+// clang-format on
 
 #include "NvDecoder/NvDecoder.h"
 #include "sora/dyn/cuda.h"

diff --git a/third_party/NvCodec/NvCodec/NvDecoder/NvDecoder.h b/third_party/NvCodec/NvCodec/NvDecoder/NvDecoder.h
@@ -15,9 +15,10 @@
 #include <stdint.h>
 #include <string.h>
 
-// https://github.com/llvm/llvm-project-release-prs/pull/698/files
-#undef __noinline__
-#include <iostream>
+// clang-format off
+#include "sora/fix_cuda_noinline_macro_error.h"
+#include <string>
+// clang-format on
 
 #include <map>
 #include <mutex>

diff --git a/third_party/NvCodec/NvCodec/NvEncoder/NvEncoder.h b/third_party/NvCodec/NvCodec/NvEncoder/NvEncoder.h
@@ -14,9 +14,10 @@
 #include <stdint.h>
 #include <string.h>
 
-// https://github.com/llvm/llvm-project-release-prs/pull/698/files
-#undef __noinline__
+// clang-format off
+#include "sora/fix_cuda_noinline_macro_error.h"
 #include <iostream>
+// clang-format on
 
 #include <mutex>
 #include <sstream>

diff --git a/third_party/NvCodec/NvCodec/NvEncoder/NvEncoderCuda.h b/third_party/NvCodec/NvCodec/NvEncoder/NvEncoderCuda.h
@@ -14,9 +14,10 @@
 #include <cuda.h>
 #include <stdint.h>
 
-// https://github.com/llvm/llvm-project-release-prs/pull/698/files
-#undef __noinline__
+// clang-format off
+#include "sora/fix_cuda_noinline_macro_error.h"
 #include <mutex>
+// clang-format on
 
 #include <vector>
 #include "NvEncoder.h"