Various cppcheck fixes (#337)

Fixed out of bounds read in AVX512VBMI version of fdr_exec_fat_teddy … (#333)

Fixed out of bounds read in AVX512VBMI version of fdr_exec_fat_teddy (#322)

  * Replaced the 32 byte read with a properly truncated mapped read
  * Added a unit test

Co-authored-by: Rafał Dowgird <rafal.dowgird@rtbhouse.com>

partial_load_u64 will fail if buf == NULL/c_len == 0 (#331)

Clang 17+ is more restrictive on rebind<T> on MacOS/Boost, remove warning (#332)

* Clang 17+ is more restrictive on rebind<T> on MacOS/Boost, remove warning

* More clang/boost warnings on MacOS, disable for now

Fix 5.4.11's config step regression (#327)

An old commit (24ae1670d) had the side effect of moving cmake defines after
they were being used. This patch move them back to be defined before being used.
Speed hsbench back up by ~ 0.8%

Signed-off-by: Yoan Picchi <yoan.picchi@arm.com>

Fix typo in build instructions (#315)

Fix regression error #317 and add unit test (#318)

Revert the code that produced the regression error in #317
Add the regression error to a unit test regressions.cpp along with the rebar tests

---------

Co-authored-by: gtsoul-tech <gtsoulkanakis@gmail.com>

Cppcheck errors fixed and suppressed (#319)

* supress knownConditionTrueFalse

* cppcheck suppress redundantInitialization

* cppcheck solve stlcstrStream

* cppcheck suppress useStlAlgorithm

* cppcheck-suppress derefInvalidIteratorRedundantCheck

* cppcheck solvwe constParameterReference

* const parameter reference cppcheck

* removed wrong fix

* cppcheck-suppress memsetClassFloat

* cppcheck fix memsetClassFloat

* cppcheck fix unsignedLessThanZero

* supressing all errors on simde gitmodule

* fix typo (unsignedLessThanZero)

* fix cppcheck suppress simde gitmodule

* cppcheck-suppress unsignedLessThanZero

---------

Co-authored-by: gtsoul-tech <gtsoulkanakis@gmail.com>

Fix noodle sve2 off by one (#313)

* Revert "Fix noodle SVE2 off by one bug"

This patch was fixing the bug when it happens at the end of the buffer
but it wasn't fixing it when we do scanDoubleOnce before the main loop

The next patch fix this bug for both case instead

This reverts commit 48dd0e5ff0bc1995d62461c92cfb76d44d1d0105.

* Fix noodle spurious match with \0 chars for SVE2

When sve2's noodle process a non full vector (before the main loop or
at the end of it), a fake \0 was being parsed, trigerring a match for
pattern that ended with \0. This patch fix this.

Signed-off-by: Yoan Picchi <yoan.picchi@arm.com>
---------

Signed-off-by: Yoan Picchi <yoan.picchi@arm.com>

hs_valid_platform: Fix check for SSE4.2 (#310)

Vectorscan requires SSE4.2 as a minimum on x86_64. For Hyperscan this
used to be SSSE3.

Applications that use the library call hs_valid_platform() to check if
the CPU fulfils this minimum requirement. However, when Vectorscan
upgraded to SSE4.2, the check was not updated. This leads to the library
trying to execute instructions that are not supported, resulting in the
application to crash.

This might not have been noticed as the CPUs that do not support SSE4.2
are rather old and unlikely to run any load where performance is an
issue. However, I believe that the library should not let the
application crash.

Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>

Make vectorscan accept \0 starting pattern (#312)

Vectorscan used to reject such pattern because they were being compared
to "" and found to be an empty string. We now check the pattern length
instead.

Signed-off-by: Yoan Picchi <yoan.picchi@arm.com>

Fix noodle SVE2 off by one bug (#309)

By using svmatch on 16 bit lanes with a 8 bit predicate, we end up
including an undefined character in the pattern checks. The inactive
lane after load contains an undefined value, usually \0. Patterns
using \0 as the last character would then match this spurious
character, returning a match beyond the buffer's end. The fix checks
for such matches and rejects them.

Signed-off-by: Yoan Picchi <yoan.picchi@arm.com>

Bug fix/rebar tests (#307)

* fixed paths and utf8-lossy=true

* revert to maskz (its the bug)

* cppcheck fix

---------

Co-authored-by: gtsoul-tech <gtsoulkanakis@gmail.com>

Rebar based Unit tests (#305)

* rebar based unit tests

* fixing paths

---------

Co-authored-by: gtsoul-tech <gtsoulkanakis@gmail.com>

maybe fix the hsbench issue (check_ssse3 again) in sse2/simde env (#306)

* maybe fix the hsbench issue (check_ssse3 again) in sse2/simde env

* fix the last failing unit test with fat

---------

Co-authored-by: G.E. <gregory.economou@vectorcamp.gr>

build/run on machines that only have SSE2 with SIMDE (#303)

This allows the use of SIMDE library to emulate SSSE3/SSE4.2 instructions on SSE2-only (x86-64-v2) hardware.

---------

Co-authored-by: G.E <gregory.economou@vectorcamp.gr>
Co-authored-by: Konstantinos Margaritis <konstantinos@vectorcamp.gr>

Teddy macros unrolling - initial PR to test in CI (#294)

Major refactoring of teddy and teddy_avx2, unrolling macros to C++ templated functions

---------

Co-authored-by: G.E <gregory.economou@vectorcamp.gr>

Bug fix/clang-tidy-performance (#300)

Various clang-tidy-performance fixes:
* noexcept
* performance-noexcept-swap
* performance
* performance-move-const-arg
* performance-unnecessary-value-param
* performance-inefficient-vector-operation
* performance-no-int-to-ptr
* add performance
* performance-inefficient-string-concatenation
* clang-analyzer-deadcode.DeadStores
* performance-inefficient-vector-operation
* clang-analyzer-core.NullDereference
* clang-analyzer-core.UndefinedBinaryOperatorResult
* clang-analyzer-core.CallAndMessage

---------

Co-authored-by: gtsoul-tech <gtsoulkanakis@gmail.com>

Script for the clang-tidy CI (#299)

script to clang-tidy CI

Co-authored-by: gtsoul-tech <gtsoulkanakis@gmail.com>

Bug fix/clang tidy warnings part3 (#298)

* clang-analyzer-deadcode.DeadStores

* clang-analyzer-optin.performance.Padding

---------

Co-authored-by: gtsoul-tech <gtsoulkanakis@gmail.com>

Clang-tidy config (#297)

clang-tidy config

Co-authored-by: gtsoul-tech <gtsoulkanakis@gmail.com>

Bug fix/clang tidy warnings part2 (#296)

* core.StackAddressEscape

* cplusplus.VirtualCall

* clang-analyzer-deadcode.DeadStores

* clang-analyzer-core.NullDereference

* clang-analyzer-core.NonNullParamChecker

* change to nolint

---------

Co-authored-by: gtsoul-tech <gtsoulkanakis@gmail.com>

Fix Clang Tidy warnings (#295)

Fixes some of the clang-tidy warnings
clang-analyzer-deadcode.DeadStores
clang-analyzer-cplusplus.NewDelete
clang-analyzer-core.uninitialized.UndefReturn

closes some:#253

ignored in this pr:
/usr/include/boost/smart_ptr/detail/shared_count.hpp:432:24
/usr/include/boost/smart_ptr/detail/shared_count.hpp:443:24
51 in build/src/parser
gtest ones
src/fdr/teddy_compile.cpp:600:5 refactoring on way
src/fdr/fdr_compile.cpp:209:5 refactoring on way

clang-analyzer-cplusplus.Move

clang-analyzer-core.uninitialized.UndefReturn

remove comment

Fix Clang Tidy warning optin.performance.Padding (#293)

Fixes some optin.performance.Padding

closes some: #253

clang-analyzer-cplusplus.NewDelete

deadcode.DeadStores

deadcode.DeadStores

uninitialized.UndefReturn

cplusplus.Move

optin.performance.Padding

Fix/Suppress remaining Cppcheck warnings (#291)

Fix/suppress the following cppcheck warnings:

* arithOperationsOnVoidPointer
* uninitMember
* const*
* shadowVariable
* assignmentIntegerToAddress
* containerOutOfBounds
* pointer-related warnings in Ragel source
* missingOverride
* memleak
* knownConditionTrueFalse
* noExplicitConstructor
* invalidPrintfArgType_sint
* useStlAlgorithm
* cstyleCast
* clarifyCondition
* VSX-related cstyleCast
* unsignedLessThanZero

Furthermore, we added a suppression list to be used, which also includes the following:
* missingIncludeSystem
* missingInclude
* unmatchedSuppression

Part 5 of C-style cast cppcheck (#289)

Fixes some cstyleCasts part 5

closes some: #252

revert uniform cstyle suppress

Speed up truffle with 256b TBL instructions (#290)

256b wide SVE vectors allow some simplification of truffle. Up to 40%
speedup on graviton3. Going from 12500 MB/s to 17000 MB/s onhe
microbenchmark.
SVE2 also offer this capability for 128b vector with a speedup around
25% compared to normal SVE

Add unit tests and benchmark for this wide variant

Speed up truffle with 256b TBL instructions

256b wide SVE vectors allow some simplification of truffle.
Up to 40% speedup on graviton3. Going from 12500 MB/s to 17000 MB/s
onhe microbenchmark.
SVE2 also offer this capability for 128b vector with a speedup around
25% compared to normal SVE

Add unit tests and benchmark for this wide variant

Signed-off-by: Yoan Picchi <yoan.picchi@arm.com>

cstylecast parser

cstylecasts and suppressions

Merge pull request #288 from isildur-g/bugfix-assert

revert a change to assert

Merge pull request #287 from gtsoul-tech/bugFix/cppcheck-cStylecasts-Part4

Part 4 of C-style cast cppcheck

Merge pull request #283 from isildur-g/wip-cppcheck271-part2

Wip cppcheck271 useStlAlgorithm part2

revert a change to assert , the original logic might have been
subtely clever (or else totally useless all these years), when we
see which of the two we might delete that assert entirely. for now
put it back as it was.

cstylecasts suppress,fixes

undo that one, it breaks

Merge pull request #286 from VectorCamp/bugfix/cppcheck-fix-wrong-casts

Fix remaining C style casts and suppressions

fix cStyleCasts

Fix typo

Add more C style casts fixes and suppressions

remove unneeded suppression

Fix more C-style casts

Fix casts

Fix wrong comparison

Fix wrong casts in pcapscan.cc

Merge pull request #285 from gtsoul-tech/bugFix/cppcheck-cStylecasts-Part3

Part 3 of C-style cast cppcheck

Merge pull request #284 from gtsoul-tech/bigFix/cppcheck-cStylecasts-Part2

Part 2 of C-style cast cppcheck

stl'ed another.

stl'ed another one

stl'ed one more instance

cStyleCasts

cStyleCasts

clean up comments

Merge branch 'develop' into wip-cppcheck271-part2

last batch

next batch

next batch

Merge pull request #281 from VectorCamp/bugfix/cppcheck-cstyleCasts-part1

Part 1 of C-style cast cppcheck fixes

first batch of cppcheck disables and a few more stl-ifications,
involving use of accumulate() .

add alternative macro without C casts to avoid Cppcheck warnings

Fix C style casts in mcsheng_compile.cpp

Merge pull request #280 from isildur-g/bugfix-rose-segfault

the segfault we ran into, included_frag_id should be

Fix wrong cast in aligned_free()

Fix C-style casts

Merge branch 'develop' into bugfix-rose-segfault

the segfault we ran into, included_frag_id should be
included_delay_frag_id.

Merge pull request #279 from VectorCamp/bugfix/cppcheck-unreadVariable-others

Fix marked as done cppcheck warnings unreadVariable & others

Merge pull request #278 from VectorCamp/bugfix/cppcheck-remaining-const-warnings

Fix remaining marked as done const* cppcheck warnings

remove unused variables

another duplicateExpression false positive

Fix unreadVariable warning

Fix missingOverride

Fix unknownMacro

Fix variableScope

False positive knownConditionTrueFalse

False positives duplicateExpression

Fix remaining marked as done const* cppcheck warnings

Merge pull request #277 from isildur-g/wip-cppcheck271

phase 1 of addressing cppcheck useStlAlgorithm warnings for fill and copy operations

Merge pull request #276 from gtsoul-tech/bugFix/cppcheck-noExplicitConstructor-part2

Bug fix/cppcheck no explicit constructor part2

phase 1 of addressing cppcheck useStlAlgorithm warnings,
this set only includes fill and copy operations.

supervector conversion

forgot some conversions SuperVector<32>

explicit constructor Supervector

ppc64el supervector explicit constructor

arm supervector explicit constructor

x86 explicit constructor supervector

Merge pull request #274 from gtsoul-tech/bugFix/cppcheckError-noexplicitConstructor

Bug fix/cppcheck error noexplicitconstructor

most ptr.get() conversion

fixes left_id and suffix_id

more fixes