For all the not yet so tech savy people, I propose a comprenhensive H…#41
Open
s9anmey2 wants to merge 2 commits intoszeged:masterfrom
Open
For all the not yet so tech savy people, I propose a comprenhensive H…#41s9anmey2 wants to merge 2 commits intoszeged:masterfrom
s9anmey2 wants to merge 2 commits intoszeged:masterfrom
Conversation
bgabor666
reviewed
Jun 27, 2017
| Use it with ./csibe.py --toolchain <myCompiler>-<myCore>. | ||
|
|
||
|
|
||
| 2. If you want add some project, add a <myProject> directory to gen/ and src/. In gen/<myProject>, put a CMakeList.txt file. Look up the existing ones for inspiration. Put the .c/.cpp files into -surprise- src/<myProject>. |
Contributor
There was a problem hiding this comment.
Please remove '-surprise-' from this line. Thanks!
bgabor666
reviewed
Jun 27, 2017
Contributor
bgabor666
left a comment
bgabor666
left a comment
There was a problem hiding this comment.
Hi s9anmey2,
Thank you for contribution to CSiBE!
I would like to suggest that you add this guide to the README.md file, appending it to the 'Basic usage' section.
Thanks,
Gabor Ballabas
Contributor
|
One more thing @s9anmey2 , please squash the commits into one! You can use the last commit message for the squashed one. |
This change is needed because the add_compile_options command, has only been available in CMake since version 2.8.12. For all the not yet so tech savy people, I propose a comprenhensive HowTo. Since I count myself to the group, I contributed what I figured out myself. Feedback would be awesome. A few hints for beginners in the readme.md
Author
|
Hi,
I would like to work with assembler files for ARM architectures, but
your benchmark does not support them yet.
Do you have a hint how to alter the script to allow for asm files?
kind regards,
Andreas
Quoting Gabor Ballabas <notifications@github.com>:
… bgabor666 commented on this pull request.
Hi s9anmey2,
Thank you for contribution to CSiBE!
I would like to suggest that you add this guide to the README.md
file, appending it to the 'Basic usage' section.
Thanks,
Gabor Ballabas
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
#41 (review)
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…owTo. Since I count myself to the group, I contributed what I figured out myself. Feedback would be awesome.