Skip to content

fix: set correct state before writing output - #1752

Open
Vanshika Turakhia (vansvan17) wants to merge 2 commits into
qualcomm:mainfrom
vansvan17:fix/act-before-writing-output-state
Open

fix: set correct state before writing output#1752
Vanshika Turakhia (vansvan17) wants to merge 2 commits into
qualcomm:mainfrom
vansvan17:fix/act-before-writing-output-state

Conversation

@vansvan17

Copy link
Copy Markdown

Summary

  • set LinkState::ActBeforeWritingOutput before invoking the corresponding plugin hook
  • strengthen the existing plugin test so it reports an error when the hook observes any other state

Testing

  • syntax-checked ELFObjectWriter.cpp with the generated external-LLVM compile command
  • syntax-checked ActBeforeWritingOutputPlugin.cpp against LLVM 22.1.8
  • external macOS build reaches C++ compilation but is blocked project-wide by the existing ELF .comment inline assembly being rejected by the Mach-O assembler

Fixes #1728

Comment thread lib/Writers/ELFObjectWriter.cpp
Signed-off-by: Vanshika <vanshikaturakhia@gmail.com>
Signed-off-by: Vanshika <vanshikaturakhia@gmail.com>
@parth-07
Parth (parth-07) force-pushed the fix/act-before-writing-output-state branch from 61484ce to a043146 Compare August 27, 2026 06:29
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.

eld incorrectly set the link state ActBeforePerformingLayout when it should instead set ActBeforeWritingOutput

2 participants