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

[CIR] support -std=gnu89 #358

Merged
merged 2 commits into from
Dec 21, 2023
Merged

[CIR] support -std=gnu89 #358

merged 2 commits into from
Dec 21, 2023

Conversation

gitoleg
Copy link
Collaborator

@gitoleg gitoleg commented Dec 19, 2023

Tiny PR, support -std=gnu89 option
This is quite frequent bug in llvm-test-suite

Copy link
Member

@bcardosolopes bcardosolopes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, please add a testcase!

@lanza lanza force-pushed the main branch 2 times, most recently from 05ffb2a to 34fceae Compare December 20, 2023 07:10
@gitoleg
Copy link
Collaborator Author

gitoleg commented Dec 20, 2023

@bcardosolopes done! added test with -std=gnu89 option

Copy link
Member

@bcardosolopes bcardosolopes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bcardosolopes bcardosolopes merged commit 901656f into llvm:main Dec 21, 2023
4 of 6 checks passed
lanza pushed a commit that referenced this pull request Jan 29, 2024
Tiny PR, support `-std=gnu89` option
This is quite frequent bug in `llvm-test-suite`
lanza pushed a commit that referenced this pull request Mar 23, 2024
Tiny PR, support `-std=gnu89` option
This is quite frequent bug in `llvm-test-suite`
eZWALT pushed a commit to eZWALT/clangir that referenced this pull request Mar 24, 2024
Tiny PR, support `-std=gnu89` option
This is quite frequent bug in `llvm-test-suite`
eZWALT pushed a commit to eZWALT/clangir that referenced this pull request Mar 24, 2024
Tiny PR, support `-std=gnu89` option
This is quite frequent bug in `llvm-test-suite`
lanza pushed a commit that referenced this pull request Apr 29, 2024
Tiny PR, support `-std=gnu89` option
This is quite frequent bug in `llvm-test-suite`
lanza pushed a commit that referenced this pull request Apr 29, 2024
Tiny PR, support `-std=gnu89` option
This is quite frequent bug in `llvm-test-suite`
eZWALT pushed a commit to eZWALT/clangir that referenced this pull request Apr 29, 2024
Tiny PR, support `-std=gnu89` option
This is quite frequent bug in `llvm-test-suite`
lanza pushed a commit that referenced this pull request Apr 29, 2024
Tiny PR, support `-std=gnu89` option
This is quite frequent bug in `llvm-test-suite`
bruteforceboy pushed a commit to bruteforceboy/clangir that referenced this pull request Oct 2, 2024
Tiny PR, support `-std=gnu89` option
This is quite frequent bug in `llvm-test-suite`
Hugobros3 pushed a commit to shady-gang/clangir that referenced this pull request Oct 2, 2024
Tiny PR, support `-std=gnu89` option
This is quite frequent bug in `llvm-test-suite`
keryell pushed a commit to keryell/clangir that referenced this pull request Oct 19, 2024
Tiny PR, support `-std=gnu89` option
This is quite frequent bug in `llvm-test-suite`
lanza pushed a commit that referenced this pull request Nov 5, 2024
Tiny PR, support `-std=gnu89` option
This is quite frequent bug in `llvm-test-suite`
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