Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

アセンブラ出力を行い、artifacts に含める #290

Merged
merged 2 commits into from
Jul 23, 2018

Conversation

m-tmatma
Copy link
Member

#286: アセンブラ出力を行い、artifacts に含める

コンパイラの設定でオブジェクトファイルの生成と同時にアセンブラ出力も行う。

構成 サイズ 圧縮にかかる時間
x64 Debug 14MB 12 秒
Win32 Debug 13MB 8 秒
x64 Release 6MB 4 秒
Win32 Release 5MB 4 秒

@m-tmatma m-tmatma added enhancement ■機能追加 CI appveyor など CI 関連 【ChangeLog除外】 labels Jul 20, 2018
@m-tmatma m-tmatma added this to the next release milestone Jul 20, 2018
@m-tmatma
Copy link
Member Author

文字コード変換によるデグレがないことを検証することが
目的の一つなので、この PRの マージ後に文字コード変換の
PR を rebase するのを想定してます。

@@ -105,6 +105,7 @@
<DebugInformationFormat>ProgramDatabase</DebugInformationFormat>
<AdditionalOptions>/source-charset:shift_jis %(AdditionalOptions)</AdditionalOptions>
<MultiProcessorCompilation>true</MultiProcessorCompilation>
<AssemblerOutput>AssemblyAndSourceCode</AssemblerOutput>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

お疲れ様です。1件お願いです。
アセンブラリストの出力にUnicodeを使う、というオプションが存在するので、それも一緒に入れたいです。
#264 (comment) ←このコメントでも書いてますが。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

今 pc 環境がないので、2日後に確認します。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

アセンブラリストの出力にUnicodeを使う、というオプションが存在するので、それも一緒に入れたいです。

これを入れると、もともとのソースコードが含まれなくなるみたいだ。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

↑ サイズが半減する。(圧縮後のサイズで 5 MB → 3MB)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ま、いいか

Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMです。

@m-tmatma m-tmatma merged commit b9ffcf3 into sakura-editor:master Jul 23, 2018
@m-tmatma m-tmatma deleted the feature/asm-output branch July 23, 2018 00:11
@kobake
Copy link
Member

kobake commented Jul 23, 2018

ご対応ありがとうございます。明日あたりUTF系PRのrebaseやっておきます

@ds14050 ds14050 added enhancement ■機能追加 CI appveyor など CI 関連 【ChangeLog除外】 labels Sep 18, 2018
takke added a commit to takke/sakura that referenced this pull request Apr 3, 2019
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
アセンブラ出力を行い、artifacts に含める
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI appveyor など CI 関連 【ChangeLog除外】 enhancement ■機能追加
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants