Skip to content
New issue

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

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

Already on GitHub? # to your account

誤記報告: P195の作成するファイルの名前、P513のテスト実行コマンドについて #11

Closed
himadajin opened this issue Aug 11, 2022 · 2 comments

Comments

@himadajin
Copy link

P195のLLVMにテストを追加する箇所でファイル名の表記揺れがありました。
「simple_test.llというファイルを追加します」という説明と、添付されたコードのファイル名は"llvm/test/CodeGen/simple_test.ll"となっていました。一方で、llvm-litのコマンドを実行する説明以降の箇所ではファイル名が"simple_func.ll"となっていました。
この2つは同じファイルを指していると思われるので、simple_func.llが正しいのではないでしょうか?

つぎに、P513のllvm-litでRISCVのすべてのテストを実行する説明において、コマンドが
${BUILD}/bin/llvm-lit -v llvm-myriscvx120/test/CodeGen/RISCV
となっていましたが、testディレクトリはllvm-myriscv120/llvm以下にあるため
${BUILD}/bin/llvm-lit -v llvm-myriscvx120/llvm/test/CodeGen/RISCV
が正しいと思います。

別の意図があったり、誤記ではなかったら申し訳ありません。

@msyksphinz-self
Copy link
Owner

ありがとうございます。ご指摘の通りです。READMEの正誤表を更新しました。

@himadajin
Copy link
Author

ご対応ありがとうございます。

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants