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

アセンブラ出力を有効にしませんか? #286

Closed
m-tmatma opened this issue Jul 19, 2018 · 3 comments
Closed

アセンブラ出力を有効にしませんか? #286

m-tmatma opened this issue Jul 19, 2018 · 3 comments
Labels
enhancement ■機能追加
Milestone

Comments

@m-tmatma
Copy link
Member

アセンブラ出力を有効にしませんか?

以下で @berryzplus さんが文字コード変換で問題がないかの確認でアセンブラ出力を活用されました。
#265 (review)

実際に生成されるコードを比較して確認すると心強いと思います。
なのでデフォルトで有効にしませんか?

昔、アセンブラ出力を行うとバイナリが生成されない仕様だったように思いますが、
15.7.5 で試すとアセンブラ出力もバイナリも生成されていました。

以下で設定できます。

assember

@m-tmatma m-tmatma added the enhancement ■機能追加 label Jul 19, 2018
@m-tmatma
Copy link
Member Author

asm ファイルも 別ファイルとして artifacts に含めようと思います。

zip 形式にすると win32 debug で 18MB ぐらい
7z 形式にすると win32 debug で 6 MB ぐらいなので
7z 形式にしようと思います。

7z 形式の欠点は (ローカルで試した限り)
zip 形式では 8 秒で済んだのに対して
7z 形式では 圧縮に 47 秒ほどかかった。

@kobake
Copy link
Member

kobake commented Jul 20, 2018

良いと思います。

形式については自分としては容量よりも圧縮時間が短いことを優先したいです。
AppVeyor 上でどの程度かかるか分かりませんが、ローカルで試してもらった数字の差を見ても、あまり無視できる時間ではないな、と思いました。成果物4つできるわけですから、4倍の時間差になるわけですよね。

@berryzplus
Copy link
Contributor

心配してるのは元ソースコードの文字コードが混在する場合にどうなるかの確認が取れてなかったことです。余分に時間かかることが分かっているなら、utf8対応の完了後にunicodeオプションは外してもいいと思っています。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ■機能追加
Projects
None yet
Development

No branches or pull requests

4 participants