Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@kbkpbot
Copy link
Contributor

@kbkpbot kbkpbot commented Aug 26, 2025

add @OS @COMPILER @BACKEND @PLATFORM comptime support.

@huly-for-github
Copy link

Connected to Huly®: V_0.6-24130

Copy link
Member

@Delta456 Delta456 left a comment

Choose a reason for hiding this comment

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

Awesome work 💯

Copy link
Member

@spytheman spytheman left a comment

Choose a reason for hiding this comment

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

Excellent work.

@spytheman spytheman changed the title checker: add @OS @COMPILER @PLATFORM v: add support for @OS, @COMPILER and @PLATFORM Aug 26, 2025
@spytheman
Copy link
Member

I am not sure if the @COMPILER name is clear enough.
V has other backends too, which makes me think that @CCOMPILER may be a better name.

@spytheman
Copy link
Member

Another possibility is to allow @COMPILER to be one of wasm, js, go, native too 🤔 .

@kbkpbot
Copy link
Contributor Author

kbkpbot commented Aug 26, 2025

Another possibility is to allow @COMPILER to be one of wasm, js, go, native too 🤔 .

Or introduce another @BACKEND ?

pub enum Backend {
        c               // The (default) C backend
        golang          // Go backend
        interpret       // Interpret the ast
        js_node         // The JavaScript NodeJS backend
        js_browser      // The JavaScript browser backend
        js_freestanding // The JavaScript freestanding backend
        native          // The Native backend
        wasm            // The WebAssembly backend
}

@kbkpbot
Copy link
Contributor Author

kbkpbot commented Aug 26, 2025

I am not sure if the @COMPILER name is clear enough. V has other backends too, which makes me think that @CCOMPILER may be a better name.

Yes, it is better rename to @CCOMPILER

@kbkpbot kbkpbot changed the title v: add support for @OS, @COMPILER and @PLATFORM v: add support for @OS, @COMPILER @BACKEND and @PLATFORM Aug 26, 2025
@spytheman spytheman changed the title v: add support for @OS, @COMPILER @BACKEND and @PLATFORM v: add support for @OS, @CCOMPILER @BACKEND and @PLATFORM Aug 27, 2025
@spytheman
Copy link
Member

I've asked @medvednikov on Discord, and he said that the current names of these sound good.

@spytheman spytheman merged commit 9e1273a into vlang:master Aug 27, 2025
83 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