OpenMP のリンク順序を v0.12.0 に戻す - #525
Merged
Merged
Conversation
v0.12.1 の Linux x86_64 wheel では libiomp5 が静的にも取り込まれて 二重に入り、実行時に SIGSEGV が起きることがあったため、OpenMP を BLAS / LAPACK より前でリンクする順序に戻す。
Contributor
There was a problem hiding this comment.
Pull request overview
OpenMP ランタイムの二重取り込みによる Linux x86_64 wheel 実行時の SIGSEGV リスクを避けるため、OpenMP のリンク順序を v0.12.0 相当に戻す PR です。OPENJIJ_USE_OMP による OpenMP の有効/無効切り替えは維持したまま、リンク順だけを調整しています。
Changes:
cxxjij_header_onlyのtarget_link_librariesにおいてOpenMP::OpenMP_CXXを BLAS/LAPACK より前に移動- OpenMP のリンク指定を
if(OPENJIJ_USE_OMP)ブロックから generator expression に移し、リンク順序をリスト内で明確化
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
OpenMP のリンク順序を v0.12.0 に戻す。
v0.12.1 の Linux x86_64 wheel では OpenMP ランタイムが二重に入り、実行時に SIGSEGV が起きることがあった。
変更内容
include/CMakeLists.txtで、OpenMP::OpenMP_CXXのリンク設定を BLAS / LAPACK より前に戻す。OPENJIJ_USE_OMPによる OpenMP の切り替えは維持する。確認したこと
Build&Upload（run 32106250890）をworkflow_dispatchで実行し、すべてのビルドが成功したcp310〜cp314）に OpenMP ランタイムが静的に取り込まれていないことを確認した