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

Clang Support #67

Open
Maroc-OS opened this issue Feb 16, 2017 · 7 comments
Open

Clang Support #67

Maroc-OS opened this issue Feb 16, 2017 · 7 comments

Comments

@Maroc-OS
Copy link

Why not thinking about adding Clang Support? the code is not clean enough, and have to be standardised, clang does not accept to compile it because of that.

Must of problems are "invalid operands to binary expression" because of mal-placements of templates/functions.

Clang is very powerful in case of code analysis and error reporting, and can help fining more bugs and fixing more problems and even optimizing the code.

Any Chances?

@chfast
Copy link

chfast commented Mar 7, 2017

I can fix issues with clang. If done, will that be merged in?

@tromer
Copy link
Member

tromer commented Mar 8, 2017

@chfast, that would be great! But please do so on top of the cmake branch, which we plan to transition to (#65).

@chfast
Copy link

chfast commented Mar 8, 2017

The cmake branch sound even better :)

@Maroc-OS
Copy link
Author

Maroc-OS commented Mar 9, 2017

i have sent a very small pr about this and i think i have some other patches.

zcash#12

@chfast
Copy link

chfast commented Mar 10, 2017

Ok, nice. Do you plan to upstream the changes here?
How much different is the zcash version?

@copumpkin
Copy link

I would love to see this, too

@MarkLTZ
Copy link

MarkLTZ commented Mar 28, 2019

I have the same issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants