Skip to content

gen: fix control-flow protection attribute marking#4828

Merged
kinke merged 2 commits intoldc-developers:masterfrom
liushuyu:fix-cf-attr-marking
Jan 23, 2025
Merged

gen: fix control-flow protection attribute marking#4828
kinke merged 2 commits intoldc-developers:masterfrom
liushuyu:fix-cf-attr-marking

Conversation

@liushuyu
Copy link
Copy Markdown
Contributor

This pull request fixes control-flow protection attribute marking by explicitly replacing the old values to avoid an LLVM assertion.

This can happen when compiling multiple D source files in one command line invocation.

... the module attribute to avoid LLVM assertions if specified module
attribute already exists
@liushuyu liushuyu force-pushed the fix-cf-attr-marking branch from c7eb5a0 to b8a9e9e Compare January 23, 2025 19:20
@kinke kinke enabled auto-merge (squash) January 23, 2025 19:26
@kinke
Copy link
Copy Markdown
Member

kinke commented Jan 23, 2025

Thx! [Btw, the PR number for the changelog ref can be guessed as max(PR/issue number) + 1.]

@liushuyu
Copy link
Copy Markdown
Contributor Author

Thx! [Btw, the PR number for the changelog ref can be guessed as max(PR/issue number) + 1.]

Thanks for the tip!

@kinke kinke merged commit c71848f into ldc-developers:master Jan 23, 2025
@liushuyu liushuyu deleted the fix-cf-attr-marking branch February 21, 2025 02:22
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.

2 participants