Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add append existing basic block method to FunctionValue impl #645

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
Merged
TheDan64 merged 24 commits into from
Jan 8, 2026
Merged

Add append existing basic block method to FunctionValue impl #645

TheDan64 merged 24 commits into from
Jan 8, 2026
+52 −0

Conversation

@marxin
Copy link
Contributor

@marxin marxin commented Jan 7, 2026

Supersedes #625.

tahadostifam and others added 24 commits November 5, 2025 20:39
@tahadostifam
feat: added is_const method to BasicValueEnum
c9f8b9c
@tahadostifam
feat: added constness check for ScalableVectorValue in BasicValueEnum…
6ae89f8 
…::is_const() which wasn't support in previous commit
@tahadostifam
feat: removed BasicValueEnum::is_const() and added BasicValue::is_con…
503e6af 
…st()
@tahadostifam
feat: changed Value construction to check is_const to call LLVMIsCons…
34320bd 
…t directly
@tahadostifam
fix: changed LLVMIsConst to LLVMIsConstant
b859049
@tahadostifam
Merge branch 'TheDan64:master' into master
d7681c1
@tahadostifam
feat: add append_existing_basic_block method to ContextImpl
52aec89
@tahadostifam
Merge branch 'master' into add_append_existing_basic_block_method_to_…
ba31b43 
…context_impl
@tahadostifam
fmt: cargo fmt
c202ddf
@tahadostifam
fix: converted BasicBlock to BasicValue in append_existing_basic_block
23fbe3d
@tahadostifam
feat: add support for append_existing_basic_block with separating llv…
6c7525e 
…m20 version of it
@tahadostifam
Merge branch 'master' into add_append_existing_basic_block_method_to_…
d8cbf4a 
…context_impl
@TheDan64
Merge branch 'master' into add_append_existing_basic_block_method_to_…
74a4d88 
…context_impl
@tahadostifam
fix: remove incorrect llvm-sys from Cargo.toml which was added mistak…
20ce508 
…enly by myself in previous commits
@tahadostifam
fix: bug fix
efdc1e2
@tahadostifam
fix: bug fixed and added tests to tests/all/test_basic_block.rs
f52e0c3
@tahadostifam
fix: bug fix
7203239
@tahadostifam
fix: bug fix
44ef2bc
@tahadostifam
fix: bug fix
6199ddc
@tahadostifam
fix: bug fix
2aaa021
@tahadostifam
fix: bug fix
d813fc1
@TheDan64
Merge branch 'master' into add_append_existing_basic_block_method_to_…
42d2dea 
…context_impl
@marxin
Merge branch 'master' into test
5dff17a
@marxin
move append_existing_basic_block to FunctionValue
06c5337
@marxin marxin mentioned this pull request Jan 7, 2026
Closed
@TheDan64 TheDan64 added this to the 0.8.0 milestone Jan 7, 2026
TheDan64
TheDan64 approved these changes Jan 8, 2026
View reviewed changes
Copy link
Owner

@TheDan64 TheDan64 left a comment

Choose a reason for hiding this comment

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

Thanks!

@TheDan64 TheDan64 merged commit ad75429 into TheDan64:master Jan 8, 2026
16 checks passed
@marxin marxin deleted the append_existing_basic_block branch January 9, 2026 04:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TheDan64 TheDan64 TheDan64 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

0.8.0

Development

Successfully merging this pull request may close these issues.

3 participants

@marxin @TheDan64 @tahadostifam