exeプロジェクトでライブラリを生成させる #856
Conversation
テストプロジェクトでexeと同じコードを使えるように、LIBコマンドでobjをまとめるようにする。 この変更により、$(IntDir)sakura_core.libが生成されるようになる。
MsBuildがクリーン時に参照する削除ファイルリストにファイル名を追加し、 生成された$(IntDir)sakura_core.libがクリーン時に削除されるようにする。
作業ついでに既存の出力成果物もクリーン対象に含めておく。 $(IntDir)と$(OutDir)に出力するファイルは、クリーン対象に含めることができる。 他のパスに出力する生成ファイルはこの方法では削除できないので別の手を考える必要がある。
berryzplus
force-pushed
the
feature/generate_static_lib_for_tests
branch
from
ccf2820 to
de7a9a2
Compare
|
#579 (comment) で言及した件ですね。そちらで書いたようにプロジェクトファイルが自分では読めず、Visual Studio に読み込ませることもできず、「テストをコンパイル済みのエディタコードとリンクします #579」をアップデートする作業も「[WIP] vs2017ソリューションにテストを組み込みたい #796」によって無用になるのでできることがありませんが、PR の目的は認めています。 |
そうです。 #796 は vsソリューションに組み込むことが主眼なので、CMakeLists.txt には手を加えていませんが、やりたいことは同じです。cmake対応はどっかでやりたいですが、まずは vc++ のGUIでテストを使えるようにするのが先決だと思って動いてます。
「元々誰がやった」的な情報を残していくかどうかについては迷いがあるんですよね。 今回のも「テストをコンパイル済みのエディタコードとリンクします #579」をベースに色々調べてみた結果でこういうPRができてきたわけで、元の発想が消えたわけじゃないと思うんです。 cmake側に関して、まだPRとかは作ってないんですけど、cmakeには 「テストをコンパイル済みのエディタコードとリンクします #579」の需要を満たすための機能 があるようです。add_library(Object Libries) を適用するにはvsソリューションに相当する部分のcmake化が必要ですが、適用すれば本体ソースとテストが参照するlibの更新を連動させられるはずです。この辺、PRを出してもらえるなら、その方がマージが速いと思うので可能なら是非。 |
Object Library は使途がよくわかりませんね。まさしく .obj のコレクションで、そのアーカイブ(.lib) とは別のもののようですが。「An object library compiles source files but does not archive or link their object files into a library.」 両者の間の微妙な違いについては、以前にもリンクを張りました>「static link について - 兼雑記」 しかし .obj のリストが公式に必要とされる状況があるのでしょうか。#579 はただの苦肉の策ですし。 |
|
この PR の動作確認は何をしてますか？ |
レビューの一環として動作確認で試してほしいことを説明してほしい、と解釈しました。
です。 |
|
失礼。パスの例示が違ってますわ。 ちゃんと書くとこんな感じ。
PRの評価としては、新規に追加した出力ファイルの生成確認ができれば十分だと思います。 実のところ、追加したライブラリを使ってテストプロジェクトのリンクが成功することころまで確認していますが、このPRを入れるにあたってそこまではみなくていいと思ってます。 |
要するに、いまやってる LIB コマンドを使う場合LinkTimeCodeGeneration時間が掛かるので、msvcのリリースビルドではlibをつくるよりobjのリストを使うほうが速度的に有利になると思います。この話は完全に別件なんですけどもｗ |
| <ItemGroup> | ||
| <SakuraCore Include="$(IntDir)*.obj" /> | ||
| </ItemGroup> | ||
| <LIB OutputFile="$(IntDir)sakura_core.lib" Sources="@(SakuraCore)" SubSystem="Windows" LinkTimeCodeGeneration="$(WholeProgramOptimization)" SuppressStartupBanner="true" /> |
There was a problem hiding this comment.
$(IntDir)sakura_core.lib ではなくて $(OutDir)sakura_core.lib だと思います。
https://docs.microsoft.com/en-us/cpp/build/reference/common-macros-for-build-commands-and-properties?view=vs-2017
によると $(IntDir) は Path to the directory specified for intermediate files. とあります。
sakura_core.lib は中間ファイルではないと思います。
There was a problem hiding this comment.
↑ テストプロジェクトの入力ファイルになるはずで、別のプロジェクトの入力になるので
出力ファイルだと思います。
There was a problem hiding this comment.
sakura_core.lib は中間ファイルではないと思います。
いいえ、中間ファイルとしての lib を生成しています。
ざっくりと言ってしまうと「拡張子がlibなだけでobjだと思ってください。」です。
LIBコマンドには色々な使い方があります。
今回はカスタムタスクを追加して中間ファイルとしてのlibを生成していますが、出力ファイルとしてのlibを生成する別なやり方もあります。出力ファイルとしてのlibには、テストで利用したいシンボル定義が含まれません。
テストプロジェクトで参照したいのは、本体プロジェクトの中間生成物です。
参照する対象はobjでもいいのですが、objを直接参照する構成にすると、テストプロジェクト側に各objの名前(約300個)を記述する羽目になります。参照を1つ書けば済むように、中間生成物としてのlibを生成しています。拡張子はlibですが中間生成物なんです。
以上がカスタムlibの出力先を中間フォルダ($(IntDir))としている理由です。
「どうしても出力ファイルにしたいんだ！」ということなら、出力ファイルとしてのlib生成(=MsBuildのImpLibタスク) のパラメータを調整して、エクスポートされないシンボルを除外しないように構成する必要が出てきます。(過去何度かトライしたことありますが、全シンボルを含んだlibを生成できたことはないっす。)
↑ テストプロジェクトの入力ファイルになるはずで、別のプロジェクトの入力になるので
出力ファイルだと思います。
上で書いた通り、テストプロジェクトに参照させたいのは、本体プロジェクトの中間ファイルです。本体プログラムとしてはエクスポートしないシンボルも参照に含めたいので、テストプロジェクトが参照するオブジェクトは中間ファイルである必要があります。
There was a problem hiding this comment.
ちょっと誤解を招く表現があるので捕捉。
テストプロジェクトが参照したいのが sakura_core/Win32/$(Configuration)/*.obj であるなら、それを直接プロジェクトに含まればいいじゃないか、という点について。
たしかうまくいかなかった（ような気がする）
#856 (comment) で描いているとおり、 LIB コマンドを使わないほうがReleaseビルド時のパフォーマンスは良いです。
というわけで、もう一回試してみます。
There was a problem hiding this comment.
あっさりイケたし・・・ 😭
このPRは一旦潰して、変更結果は #796 に混ぜ込みます。
$(IntDir)sakura_core.lib の生成が不要になるので、クリーン時処理は別件になるかな。
| @@ -1007,4 +1007,20 @@ | |||
| <Import Project="$(VCTargetsPath)\Microsoft.Cpp.targets" /> | |||
| <ImportGroup Label="ExtensionTargets"> | |||
| </ImportGroup> | |||
| <Target Name="MakeSakuraCoreLib" AfterTargets="Link" Inputs="@(ClCompile)" Outputs="$(IntDir)sakura_core.lib"> | |||
| </Target> | ||
| <Target Name="AppendCleanTargets" BeforeTargets="CoreClean"> | ||
| <ItemGroup> | ||
| <DeleteOnClean Include="$(IntDir)sakura_core.lib" /> |
目的
テストプロジェクトから利用するために、sakura.exeのobj群でライブラリを作るようにします。
変更内容
関連PR
#796(vs2017ソリューションにテストを組み込みたい)
本来は他の変更と合わせて#796に入れるつもりでしたが、このPRの変更内容だけでもかなりお腹いっぱいになれるボリュームがあると感じたので、分割することにしました。