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

minimal ts conv of llvm-ir #4477

Merged
merged 3 commits into from
Dec 24, 2022
Merged

minimal ts conv of llvm-ir #4477

merged 3 commits into from
Dec 24, 2022

Conversation

partouf
Copy link
Contributor

@partouf partouf commented Dec 20, 2022

untested as i changed this on windows

(npm run format really makes a mess of things on windows O_o but adding endOfLine: 'crlf', to the .prettierrc.js fixes that)

@partouf
Copy link
Contributor Author

partouf commented Dec 20, 2022

@partouf
Copy link
Contributor Author

partouf commented Dec 23, 2022

tested locally but seems to work

@partouf partouf marked this pull request as ready for review December 23, 2022 20:27
Copy link
Member

@jeremy-rifkin jeremy-rifkin left a comment

Choose a reason for hiding this comment

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

Looks good to me

@partouf partouf merged commit 0723662 into main Dec 24, 2022
@partouf partouf deleted the llvmir2ts branch December 24, 2022 18:45
mattgodbolt pushed a commit that referenced this pull request Jan 4, 2023
mattgodbolt pushed a commit that referenced this pull request Jan 24, 2023
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.

None yet

2 participants