Skip to content

Implement extern(C) mangling for variables.#214

Open
redstar wants to merge 1 commit intosnazzy-d:masterfrom
redstar:extcvar
Open

Implement extern(C) mangling for variables.#214
redstar wants to merge 1 commit intosnazzy-d:masterfrom
redstar:extcvar

Conversation

@redstar
Copy link
Contributor

@redstar redstar commented Nov 18, 2017

extern(C) mangling is useful for interfacing with C. It enables

extern(C) int bar;

to compile.

extern(C) mangling is useful for interfacing with C. It enables

    extern(C) int bar;

to compile.
@deadalnix
Copy link
Contributor

Look good overall, however, it doesn't looks like the test is testing anything.

@redstar
Copy link
Contributor Author

redstar commented Nov 29, 2017

A lit test is required to really check this. I still need to figure out how to run lit....

@deadalnix
Copy link
Contributor

You can find existing lit tests in test/llvm

You can run them via make littest

@deadalnix deadalnix force-pushed the master branch 15 times, most recently from 8c71eeb to 6c2cf7e Compare April 15, 2018 23:38
@deadalnix deadalnix force-pushed the master branch 11 times, most recently from abb159a to ddd472c Compare April 13, 2021 17:05
@deadalnix deadalnix force-pushed the master branch 2 times, most recently from 5027160 to 6d58784 Compare June 6, 2022 01:08
@deadalnix deadalnix force-pushed the master branch 2 times, most recently from 9fcc9e5 to a1ef70f Compare September 30, 2022 17:26
@deadalnix deadalnix force-pushed the master branch 4 times, most recently from dc3ea09 to 1f71c2a Compare December 1, 2022 00:10
@deadalnix deadalnix force-pushed the master branch 2 times, most recently from 0fb9de0 to 208c449 Compare January 2, 2023 16:27
@deadalnix deadalnix force-pushed the master branch 2 times, most recently from f94baa8 to 87a9baf Compare August 23, 2023 22:14
@deadalnix deadalnix force-pushed the master branch 3 times, most recently from cf07b66 to 39f39ba Compare January 3, 2024 14:18
@deadalnix deadalnix force-pushed the master branch 2 times, most recently from 4acd1ab to e854d3d Compare June 17, 2024 21:01
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