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.

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

🔧 Change opt target #32

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

🔧 Change opt target #32

wants to merge 1 commit into from

Conversation

trickypr
Copy link
Member

@trickypr trickypr commented Jan 3, 2023

@trickypr
🔧 Change opt target
bece486 
Changes the instruction set target down to `nehalem` and the tuning architecture up to zen3 (not sure how widely available zen4 tuning is within gcc, it probably isn't available on ubuntu yet)
@trickypr trickypr requested a review from PressJump January 3, 2023 05:37
@trickypr trickypr self-assigned this Jan 3, 2023
PressJump
PressJump approved these changes Jan 3, 2023
View reviewed changes
Copy link
Member

@PressJump PressJump left a comment

Choose a reason for hiding this comment

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

LGTM!

@splatboydev
Copy link
Contributor

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PressJump PressJump PressJump approved these changes

Assignees

@trickypr trickypr

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@trickypr @splatboydev @PressJump