Skip to content
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

Compilation: Add driver/frontend support for pic/pie options #768

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

ehaas
Copy link
Collaborator

@ehaas ehaas commented Sep 8, 2024

Some time (real soon now™) I'd like to start writing a simple AST -> x86_64 assembly backend. No IR or optimizations or anything, just something that generates readable/debuggable assembly.

Is that something that you think should be part of aro? Or a separate project that uses aro as a dependency? If it's part of aro I'd want to put it behind a -fsimple-asm or similar type of flag.

Copy link
Owner

@Vexu Vexu left a comment

Choose a reason for hiding this comment

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

Some time (real soon now™) I'd like to start writing a simple AST -> x86_64 assembly backend. No IR or optimizations or anything, just something that generates readable/debuggable assembly.

Is that something that you think should be part of aro? Or a separate project that uses aro as a dependency? If it's part of aro I'd want to put it behind a -fsimple-asm or similar type of flag.

Absolutely, any kind of functional codegen would be nice. And just -S is fine as far as flags go.

@ehaas
Copy link
Collaborator Author

ehaas commented Sep 9, 2024

-S for assembly output makes sense; I was just thinking it would be nice if the toolchain could assemble and link the output; but I didn't want it to interfere with the existing Ir codepath. Maybe something like -O0 for it? So -O0 without -S would generate assembly to a temp file, invoke as, and then add the output to the link objects?

@Vexu
Copy link
Owner

Vexu commented Sep 9, 2024

Right, didn't think of that, using -O0 sounds good.

@ehaas ehaas merged commit 3ab8111 into Vexu:master Sep 9, 2024
3 checks passed
@ehaas ehaas deleted the pic branch September 9, 2024 22:42
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