Skip to content

[CodeGen][NPM] Port StackFrameLayoutAnalysisPass to NPM #130070

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

optimisan
Copy link
Contributor

No description provided.

@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_machinesanitizerbinarymetadata_to_npm branch from 003ff87 to 5c5dde0 Compare March 7, 2025 10:12
@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_stackframelayoutanalysispass_to_npm branch 2 times, most recently from dc9e5f0 to d4ed8c0 Compare March 7, 2025 10:16
@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_machinesanitizerbinarymetadata_to_npm branch from 5c5dde0 to ca3edf5 Compare March 10, 2025 04:45
@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_stackframelayoutanalysispass_to_npm branch from d4ed8c0 to 92a1774 Compare March 10, 2025 04:45
@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_machinesanitizerbinarymetadata_to_npm branch from ca3edf5 to 2a60a24 Compare March 10, 2025 05:22
@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_stackframelayoutanalysispass_to_npm branch from 92a1774 to 94618bb Compare March 10, 2025 05:22
@optimisan optimisan marked this pull request as ready for review March 10, 2025 05:31
Copy link
Collaborator

@cdevadas cdevadas left a comment

Choose a reason for hiding this comment

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

No tests?

@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_machinesanitizerbinarymetadata_to_npm branch from 2a60a24 to 17c20b1 Compare March 11, 2025 09:51
@optimisan
Copy link
Contributor Author

No tests?

Couldn't find a standalone test, CodeGen/X86/stack-frame-layout-remarks.ll will need the complete pipeline till this pass.

@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_stackframelayoutanalysispass_to_npm branch from 94618bb to 9aaee4f Compare March 11, 2025 09:58
@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_machinesanitizerbinarymetadata_to_npm branch from 17c20b1 to 0bc54f7 Compare April 14, 2025 11:42
Base automatically changed from users/optimisan/03-06-_codegen_npm_port_machinesanitizerbinarymetadata_to_npm to main April 14, 2025 15:22
@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_stackframelayoutanalysispass_to_npm branch from 9aaee4f to e349243 Compare April 15, 2025 04:32
@optimisan optimisan merged commit a388395 into main Apr 15, 2025
9 of 11 checks passed
@optimisan optimisan deleted the users/optimisan/03-06-_codegen_npm_port_stackframelayoutanalysispass_to_npm branch April 15, 2025 07:07
var-const pushed a commit to ldionne/llvm-project that referenced this pull request Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants