-
Notifications
You must be signed in to change notification settings - Fork 13
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
Feature request - support for passing compiler options #3
Comments
Hi, |
Yeah, looking at godbolt's solution would be my suggestion too.
An alternative might be to have a few selectable options like -march=native
etc, like you do for optimization level.
Thanks!
…On Mon, Aug 28, 2017 at 11:30 AM, Fred Tingaud ***@***.***> wrote:
Hi,
I thought about it, but the problem with passing compiler options like
that is that it would require some serious sanitizing, to avoid injections.
I am not very clear on how I could do it. I will have a look at godbolt's
solution and see if I can use it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#3 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA0hpLgFb4pTSffAHADLyNPdHr9ezi-_ks5scwc0gaJpZM4PD3xZ>
.
|
Yes, being able to select architecture would help in benchmarking SIMD intrinsic code! |
Hi, -march option would be extremely useful. I make a benchmarking with SIMD instructions and autovectorization |
Hi, |
Hi, |
Well, like somebody in the quick-bench-back-end issue already pointed out, this is not completely true. You do have certain guarantees:
So the minimum guaranteed is Haswell which means from the hardware side you have support for MMX, AES-NI, CLMUL, FMA3, SSE, SSE2, SSE3, SSSE3, SSE4, SSE4.1, SSE4.2, AVX up to AVX2. Broadwell did not change that, only with Skylake would we get AVX-512. They do have a footnote saying
so not sure what is up with that. You could include a runtime check for that. But either way up until SSE4.2 you are fine. This would really make quickbench so much more useful to me! |
There is workaround, if you use |
Awesome, thank you @Yankes - that seems to work. here are links to the documentation of the target attribute: and here are the most relevant values:
|
Thanks for your comments! |
Edit: Added in 5bd1fe1 -- thank you! |
As far as I see the topic is pretty old, but still relevant. |
You could always check a preset list of compiler flags. |
@FredTingaud As a stop-gap, you could allow passing compiler options when run locally for those that need. Would that be difficult? |
Thanks for the great project. It would be great if we could pass compiler options for our benchmark like gcc.godbolt.org. Right now I want to run my benchmark with -mavx2 and I can't find a way to do that.
The text was updated successfully, but these errors were encountered: