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

Skip to content

Conversation

JamesMolson
Copy link
Contributor

No description provided.

@vkbo
Copy link
Member

vkbo commented Jun 14, 2019

Is the clang bit only valid for Apple, or can we have it as a general section?

@JamesMolson
Copy link
Contributor Author

Apple only, normal clang explodes.

@vkbo
Copy link
Member

vkbo commented Jun 14, 2019

COVERAGE and GPROF flags don't apply either I assume?

Edit: COVERAGE does, GPROF does not.

@JamesMolson
Copy link
Contributor Author

gprof for sure not.

@vkbo
Copy link
Member

vkbo commented Jun 14, 2019

Checked the documentation. The Coverage flags are fine, but -pg does not exist.

@JamesMolson
Copy link
Contributor Author

um, they are not in the man pages

@vkbo
Copy link
Member

vkbo commented Jun 14, 2019

https://clang.llvm.org/docs/ClangCommandLineReference.html#cmdoption-clang-fprofile-arcs

Assuming this is the correct documentation?

@JamesMolson
Copy link
Contributor Author

ok, it does work, but the fortran part explodes with it enabled so I can't test it. -pg explodes.

AppleClang != Clang. They forked it a long time ago and have done terrible dark things to it since.

@vkbo
Copy link
Member

vkbo commented Jun 14, 2019

Ah, I see. In that case.

@vkbo
Copy link
Member

vkbo commented Jun 14, 2019

Could you please just delete the flags that do not apply? Commented out code that doesn't work anyway is just clutter ...

@vkbo vkbo merged commit d17a141 into SixTrack:master Jun 14, 2019
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