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

アセンブラ出力の指定方式を変更する #871

Merged

Conversation

berryzplus
Copy link
Contributor

目的

ビルド時に毎回以下のようなD9025警告が出ているのを出ないようにします。

1>cl : Command line warning D9025: overriding '/FAs' with '/FAu'

この事象は以前から sakura-editor/management-forum#24 で検討していた CPPTaskのバグと思われる事象 と同じなので、警告が出ている原因等の詳しいことは issue を見てください。

vs2019で対策されることを期待していましたが、直らなかったのでこちら(=利用側)で対策して警告出さないようにしてしまいたいと考えています。

対処内容

sakura.vcxproj内の以下のアセンブラ出力オプションを変更します。

タグ 変更前 変更後 コマンドライン
AdditionalOptions (なし)* /FAsu /Fa$(IntDir) /FAsu /Fa$(IntDir)
AssemblerOutput AssemblyAndSourceCode (削除) /FAs /Fa$(IntDir)
UseUnicodeForAssemblerListing true (削除) /FAu
  • AdditionalOptionsの変更前を (なし) としているのは、「アセンブラ関係のオプション指定はない」という意味です。

確認方法

PRのブランチをビルドして警告の有無を確認できればOKと考えています。

1.この PR 適用前のブランチ(master)をcheckoutする
1.この PR 適用前のブランチ(master)をビルドする ⇒ 上記警告が出力される
1.この PR のブランチをcheckoutする
1.この PR のブランチをビルドする ⇒ 上記警告が出力されなくなっている

  • 警告が出なくなっているのは appveyor/Azule Pipelinesでも確認できます。

VC++のGUIから設定すると、/FAs(アセンブラとコードを出力)と/FAu(Unicodeで出力)の2オプションが生成されてD9025警告になってしまうので、追加のオプションに/FAsuを指定する方式に変える。
@m-tmatma
Copy link
Member

Win32 Debug/Release, x64 Debug/Release で
db775c6 (対応前) と e4f9d8f (対応後) で CDlgAbout.asm を除いて、
コメント、空行以外に関して asm 出力が一致しているのを確認しました。

@m-tmatma m-tmatma added this to the v2.4.0 milestone Apr 28, 2019
@berryzplus
Copy link
Contributor Author

レビューありがとうございます。
マージしちゃいます。

@berryzplus berryzplus merged commit f9e81cb into sakura-editor:master Apr 28, 2019
@berryzplus berryzplus deleted the feature/workaround_for_D9025 branch April 28, 2019 09:55
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
…nd_for_D9025

アセンブラ出力の指定方式を変更する
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants