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

Modified TODO chapter content #38

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yulong18
Copy link

I deleted these options because I found it already done.

@asb
Copy link
Contributor

asb commented Jul 12, 2023

Getting rid of the TODO section and moving anything remaining to issues makes sense to me. I'm not sure all the listed items are covered though - e.g. explicit-relocs, -mdiv etc.

@yulong18
Copy link
Author

Yes, I'm sure. You can see code in riscv.opt and know about these options in the manual. @asb

@asb
Copy link
Contributor

asb commented Jul 20, 2023

Right, but this repo is meant to cover documentation for options that are shared between GCC and Clang. I'm not fundamentally opposed to pointing to documentation for clang or GCC for these options, but we should do this in a case-by-case basis, and it's probably better overall to just write something reasonable in this doc.

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