Skip to content

Enable runtime for clang - #2764

Open
theComputeKid wants to merge 1 commit into
premake:masterfrom
theComputeKid:clang
Open

Enable runtime for clang#2764
theComputeKid wants to merge 1 commit into
premake:masterfrom
theComputeKid:clang

Conversation

@theComputeKid

Copy link
Copy Markdown
Contributor

These are clang's equivalent to MSVC's /MT, /MTd, /MD and /MDd on Windows. Both compiler and linker flags are required.

@mercury233

Copy link
Copy Markdown
Contributor

Premake supports omitting the runtime directive and automatically inferring it, but the newly added staticruntime change does not take effect when runtime is omitted.
https://github.com/mercury233/premake-core/actions/runs/31134127921/job/92729620718?pr=4

@theComputeKid

Copy link
Copy Markdown
Contributor Author

@mercury233 : It would be easier if runtime and staticruntime were just one flag with 4 options. I agree with your comment and I have tried to incorporate it in my change, though it does make it messier to read. Could you check if the logic is correct?

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