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

Suggestion: include OrangeC in compiler-explorer #950

Open
GitMensch opened this issue Aug 14, 2023 · 3 comments
Open

Suggestion: include OrangeC in compiler-explorer #950

GitMensch opened this issue Aug 14, 2023 · 3 comments

Comments

@GitMensch
Copy link
Contributor

It is nice to be able to compare the results of compilers, and also gives the project some additional visibility.

Adding OrangeC to compiler-explorer would mean to add it to its main repo into the existing C and C++ definitions, similar to

https://github.com/compiler-explorer/compiler-explorer/blob/main/lib/compilers/zig.ts
https://github.com/compiler-explorer/compiler-explorer/blob/main/lib/compilers/zigcc.ts

https://github.com/compiler-explorer/compiler-explorer/blob/main/etc/config/c%2B%2B.amazonwin.properties
https://github.com/compiler-explorer/compiler-explorer/blob/main/etc/config/c.amazonwin.properties

and also to integrate it to the infrastructure repository, see

https://github.com/compiler-explorer/infra/blob/main/bin/yaml/zig.yaml.

Note: before this is done it is likely useful to ensure that the codegen works nicely for the examples, including the optimized ones, those are available at https://github.com/compiler-explorer/compiler-explorer/tree/main/examples/c
https://github.com/compiler-explorer/compiler-explorer/tree/main/examples/c++

After that it is likely a good to create a new compiler issue there and discuss the necessary steps and if they may could do part of the actual work.

@LADSoft
Copy link
Owner

LADSoft commented Aug 15, 2023

this sounds interesting... thank you.

@LADSoft
Copy link
Owner

LADSoft commented Sep 25, 2023

I will probably start evaluating the examples next weekend...

@LADSoft
Copy link
Owner

LADSoft commented Sep 29, 2023

ok so i looked at the code gen... as always there could be improvements but they are minor and I'm not making them now.

but i did check that the /sxx compiler switch still works and that aligned_alloc works as desired.... hadn't checked either of those in a while.

i noticed that the c++ 'alignas' didn't work the way I thought it should and will make a separate issue for that...

The only change I made for this is to add the __restrict keyword.

Next time I do a release I will submit orangec to this site...

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

No branches or pull requests

2 participants