Skip to content

Conversation

@jorickert
Copy link

No description provided.

abhishek-kaushik22 and others added 9 commits January 8, 2025 16:49
Update maintainers for inlining, and add section for NewPM/CGSCC.
)

In PR llvm#112341, the `APAS` instruction was added as part of the Armv9.6-A
specification, but it didn't take the Xt register parameter. This change
fixes this.
This can enable clang to detect dangling assignment issues, see llvm#120698.
…#121956)

Following-up on [comments
](llvm#62432 (comment))
in the issue that motivated this extension.
This does add a little computational complexity because now every
freemem operation has to be tested for every allocation. This could be
improved with some more memoisation but I think it is easier to read
this way. Let me know if you would prefer me to change this to
pre-compute the normalised addresses each freemem operation is using.

Weirdly, this change resulted in a verifier failure for the fir.declare
in the previous test case. Maybe it was previously removed as dead code
and now it isn't. Anyway I fixed that too.
@jorickert jorickert changed the title [AutoBump] Merge with 67efbd0b (Jan 08) (31) [AutoBump] Merge with 67efbd0b (Jan 08) (31) [Only tested MLIR] Mar 18, 2025
@jorickert jorickert merged commit 333f44b into bump_to_c1d01b2f Apr 14, 2025
5 checks passed
@jorickert jorickert deleted the bump_to_67efbd0b branch April 14, 2025 07:44
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.

10 participants