-
Notifications
You must be signed in to change notification settings - Fork 101
jp devel repository convention
Ryota Shioya edited this page Jan 18, 2020
·
4 revisions
現在,一部テスト関係のコードが機能していないため,make test 系が失敗する可能性があります(2020/01/18).近日中にテスト用ベンチマークを統合して修正する予定です.
-
git リポジトリで作業をする際は,github-flow に従ってください
- git-flow ではないので注意
-
要するに
- master から説明的なブランチを作成
- その上で作業
- master に対し pull request
- マージ
-
注意
- master 上で直接作業をしない
- マージ後は基本的にブランチは削除
-
git の操作に関しては,(なれてない人は)GUI クライアントの使用を強く推奨
- git のブランチの視覚的なイメージがないと,何が行われているのかが理解できない
- コミット・メッセージは Conventional Commits に従って書いてください
- 一部の古いコミットは,この規約にしたがっていません
- 以下では RSD/master にプルリクエストを送ってマージするまでにチェックすべき項目について記します.
- 別途, 下記を参照してください.
-
modelsimでのコンパイルが通る
- 対応するコマンド:
make
またはmake rtl
- 対応するコマンド:
-
簡単な機能シミュレーションテストが正しく実行される
- 対応するコマンド:
make test-build
の後にmake test-1
- テストプログラムのビルド方法や内容は テスト・プログラム 参照
- 対応するコマンド:
-
Coremark の機能シミュレーションテストが正しく実行される
- 対応するコマンド:
make test-Coremark
- RSD レポジトリには Coremark のテストコードは含まれていないので, RSD_ENV を設定して RSD-env レポジトリからコピーする必要あり(Src/Verification/TestCode/C/Makefile.inc 参照)
- 対応するコマンド:
-
Coremark の post-synthesis シミュレーションが正しく実行される
- 実行方法は QuestaSim で合成結果をシミュレーション 参照
- 実行結果の確認は Src/Verification/TestCode/C/Coremark/serial.out.txt が 同フォルダserial.ref.txt と一致しているかで行う
前節のチェックリストをクリアしたのち, github のプルリクエストを介して master にマージするようにしましょう.