-
Notifications
You must be signed in to change notification settings - Fork 103
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
Dl issue 182 #573
Dl issue 182 #573
Conversation
…ired to ensure the wrapper is working properly
return ids[n];} | ||
|
||
#define AVX_STRID_2(n) do {\ | ||
static const int ids[9] = {\ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The same as above. Need to use ids[10]
or ids[]
as there are 10 elements in array.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in latest commit
… macros in avx_routines.h;\n minor bugfixes;
The code doesn't comply our coding style, please adjust the style. |
What's going on in |
} | ||
|
||
fseek(source, 0, SEEK_END); | ||
size_t len = ftell(source); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
usually fstat(2)
is used to get file size. Moreover, since you set file pointer using fseek()
following fread()
doesn't read any data and your test doesn't test anything.
Poorly done |
Tempesta parser was cut from main source tree and supporting code was refactored for testing in standalone environment. Tempesta parser was patched with AVX insertions in performance critical places, but otherwise remains intact and can be transferred back to it's original position.