Skip to content

support new first-class denormal attributes#1313

Merged
nunoplopes merged 2 commits intoAliveToolkit:masterfrom
regehr:denorm-attr-fix
Apr 27, 2026
Merged

support new first-class denormal attributes#1313
nunoplopes merged 2 commits intoAliveToolkit:masterfrom
regehr:denorm-attr-fix

Conversation

@regehr
Copy link
Copy Markdown
Contributor

@regehr regehr commented Apr 24, 2026

in Feb Matt redid FP denorm attributes so that they're real LLVM attrs instead of strings:
llvm/llvm-project@2502e3b7badc

so alive2 now has some false alarms due to failing to parse the new-style attributes: #1308

we don't want to remove the old string-based code path / tests yet since it looks like the text IR parser does not run the IR auto-upgrader. we can remove this code path / tests once LLVM drops support for the old string attribute syntax.

if @arsenm happens to have time to take a quick look at the test cases attached to this PR, that would be helpful

@arsenm
Copy link
Copy Markdown
Contributor

arsenm commented Apr 24, 2026

once LLVM drops support for the old string attribute syntax.

It was dropped, it's only handled for bitcode autoupgrade

@nunoplopes nunoplopes merged commit cb9fbd5 into AliveToolkit:master Apr 27, 2026
16 checks passed
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