Skip to content

Conversation

@djtodoro
Copy link
Collaborator

@djtodoro djtodoro commented Aug 7, 2023

No description provided.

@djtodoro djtodoro marked this pull request as draft August 7, 2023 14:27
@djtodoro djtodoro force-pushed the feature/disable-balc-stubs branch from 9616713 to 11e268d Compare August 8, 2023 09:58
@djtodoro djtodoro marked this pull request as ready for review August 8, 2023 09:58
@djtodoro djtodoro force-pushed the feature/disable-balc-stubs branch 2 times, most recently from 0deae01 to b64f8da Compare August 8, 2023 12:16
@farazs-github
Copy link
Member

See binutils/pr2 for assembler fix and NOTRAMP relocations

Copy link
Member

@farazs-github farazs-github left a comment

Choose a reason for hiding this comment

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

Changes requested.

@djtodoro
Copy link
Collaborator Author

See binutils/pr2 for assembler fix and NOTRAMP relocations

I guess MediaTek-Labs/binutils-gdb#3

@djtodoro djtodoro force-pushed the feature/disable-balc-stubs branch from b64f8da to 2225b09 Compare August 14, 2023 08:58
Copy link
Member

@farazs-github farazs-github left a comment

Choose a reason for hiding this comment

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

LGTM. Holding off the merge till we have one round of integrated testing with all components.

@djtodoro
Copy link
Collaborator Author

djtodoro commented Jun 4, 2024

@AndrijaSyrmia cc; please make sure you port this to LLVM 16

@djtodoro djtodoro changed the base branch from nanomips to mtk/nanomips-llvm16 June 25, 2024 13:07
Added an option to disable BALC stubs optimization from llc. Added a test case for this change, changed addr.ll test as it was failing due to this change.
@AndrijaSyrmia AndrijaSyrmia force-pushed the feature/disable-balc-stubs branch from 2225b09 to 7ce1d2e Compare June 28, 2024 11:12
@farazs-github farazs-github deleted the branch MediaTek-Labs:mtk/nanomips-llvm16 July 11, 2024 16:47
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.

3 participants