Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Fix clang build error when enabling NODEBASH_STATS#10933

Merged
BruceForstall merged 1 commit intodotnet:masterfrom
hqueue:ryujit/enable_nodebash_stats
Apr 19, 2017
Merged

Fix clang build error when enabling NODEBASH_STATS#10933
BruceForstall merged 1 commit intodotnet:masterfrom
hqueue:ryujit/enable_nodebash_stats

Conversation

@hqueue
Copy link
Member

@hqueue hqueue commented Apr 13, 2017

After enabling NODEBASH_STATS in Linux, following error occurs when building CoreCLR with clang.

/opt/code/github/hqueue/coreclr/src/jit/gentree.cpp:453:8: error: C++ requires a type specifier for all declarations
inline hashme(genTreeOps op1, genTreeOps op2)
~~~~~~ ^

@hqueue
Fix clang build error when enabling NODEBASH_STATS
3efd028 
After enabling NODEBASH_STATS in Linux, following error occurs
when building coreclr with clang.

Error message: "C++ requires a type specifier for all declarations"

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
@hqueue
Copy link
Member Author

hqueue commented Apr 13, 2017

@dotnet-bot test Windows_NT x64 Debug Build and Test

@BruceForstall BruceForstall merged commit 918dbb4 into dotnet:master Apr 19, 2017
@karelz karelz modified the milestone: 2.0.0 Aug 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@BruceForstall BruceForstall Awaiting requested review from BruceForstall

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

2.0.0

Development

Successfully merging this pull request may close these issues.

4 participants

@hqueue @karelz @dnfclas @BruceForstall