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

Initial LLVM-18 support #220

Closed
wants to merge 1 commit into from
Closed

Initial LLVM-18 support #220

wants to merge 1 commit into from

Conversation

thoughtafter
Copy link
Collaborator

@thoughtafter thoughtafter commented Feb 21, 2024 •
edited

@thoughtafter thoughtafter self-assigned this Feb 21, 2024
@thoughtafter
Initial LLVM-18 support
77db8da 
preserve const manipulations of reals and ints despite deprecation of const trunc / ext

linting

improve handling of int overflows
Changes behavior: int truncate overflow is now poison rather than random integer

pass builder gets merge_functions option
add tests for inliner behavior

add tests and fix bugs for disable_always_inline

do not use inliner_threshold as necessary for inliner

disable DoubleNegation Rubocop

remove:

    LLVMConstAnd

    LLVMConstOr

    LLVMConstLShr

    LLVMConstAShr

    LLVMConstZExt

    LLVMConstSExt

    LLVMConstZExtOrBitCast

    LLVMConstSExtOrBitCast

    LLVMConstIntCast

    LLVMConstFPTrunc

    LLVMConstFPExt

    LLVMConstFPToUI

    LLVMConstFPToSI

    LLVMConstUIToFP

    LLVMConstSIToFP

    LLVMConstFPCast

cleanup pass_builder_test related to inlining

fix name

nits

test unknown pass error

dispose of passbuilderoptions

update documentation

cleanup value

restore const math for reals

refactor to_i to to_si and to_ui

cleanup and documentation

add many more tests

restore const real rem
@thoughtafter
Copy link
Collaborator Author

Given the delays on brew publishing 8.1.2, I'm wondering if I should pull out the Mac CI and ship.

@thoughtafter thoughtafter mentioned this pull request Apr 6, 2024
Merged
@thoughtafter
Copy link
Collaborator Author

Closing in favor of #237 and #238

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

@thoughtafter thoughtafter

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@thoughtafter