8000 Model solution updated · Issue #584 · kaist-cp/cs420 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Model solution updated #584

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.

Sign up for GitHub

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

Jump to bottom
Open
Jaewookim08 opened this issue Apr 7, 2025 · 3 comments
Open

Model solution updated #584

Jaewookim08 opened this issue Apr 7, 2025 · 3 comments
Labels
announcement

Comments

@Jaewookim08
Copy link
Collaborator

Thanks to @7dy109 (#580), we identified and fixed an error in our model solution: commit. The model solution now generates less unwanted code for & and compound assignment operators.

We also added the example side-effect.c to verify the fix, but this file will not be used for grading this semester. Your scores will remain unchanged.

Relevant examples/ir files have been updated. The model solution in GG will be updated soon.
@Jaewookim08 Jaewookim08 mentioned this issue Apr 7, 2025
Closed
@Lee-Janggun
Copy link
Member

The model solution in GG is also updated.

@aaarunan
Copy link
aaarunan commented Apr 11, 2025 •
edited
Loading

There is a small typo in the exclusion list:

kaist-cp/kecc-public@32283f2#r155162770
7EB0

@Lee-Janggun
Copy link
Member

@aaarunan Thank you. Fixed in kaist-cp/kecc-public@ad7b1ae.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
announcement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Jaewookim08 @Lee-Janggun @aaarunan
0