-
Notifications
You must be signed in to change notification settings - Fork 13.5k
[Driver] Reject -mcmodel=tiny on X86 #125643
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
Merged
Merged
Changes from 24 commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
0aebcd7
Adding diagnostics for unsupported option
ShashwathiNavada f791b1d
minor changes
ShashwathiNavada 689dc3a
minor changes
ShashwathiNavada 28fcb0e
minor changes
ShashwathiNavada ba6c662
Merge branch 'main' into mcmodel_crash
ShashwathiNavada 843d4cc
Addressed build fail
ShashwathiNavada adf4236
Merge branch 'llvm:main' into mcmodel_crash
ShashwathiNavada cb512f3
Added tests and releasenotes entry
ShashwathiNavada b2724ba
Merge branch 'main' into mcmodel_crash
ShashwathiNavada ad5c8ca
Merge branch 'main' into mcmodel_crash
ShashwathiNavada 5b9ff44
Merge branch 'main' into mcmodel_crash
ShashwathiNavada 0fa223c
Added testcase in mcmodel.c file
d85fad3
Merge branch 'main' into mcmodel_crash
ShashwathiNavada ca4d129
Merge branch 'main' into mcmodel_crash
ShashwathiNavada fcbc340
Merge branch 'main' into mcmodel_crash
ShashwathiNavada d2cf187
Merge branch 'llvm:main' into mcmodel_crash
ShashwathiNavada 8e2b440
Suggested changes completed
12e4423
Merge branch 'main' into mcmodel_crash
ShashwathiNavada 215aa42
Update Driver.cpp
ShashwathiNavada 39e7667
Merge branch 'llvm:main' into mcmodel_crash
ShashwathiNavada 2024997
Merge branch 'llvm:main' into mcmodel_crash
ShashwathiNavada 075a921
Removed check from driver.cpp and corrected existing one in CommonArg…
742ea57
removed unnecessary test
59985ea
minor change
0c14a0c
Update ReleaseNotes.rst
ShashwathiNavada File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The commit message/description and the note should be changed. Only X86 is affected. This change is pretty niche and does not deserve a release note.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MaskRay, Thank you for the response. I have removed the release note entry and updated the commit message as well.