Skip to content
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

Bench/refactor flags #1020

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

grisumbras
Copy link
Member

No description provided.

@cppalliance-bot
Copy link

An automated preview of the documentation is available at https://1020.jsondocs.prtest.cppalliance.org/libs/json/doc/html/index.html

Copy link

codecov bot commented Jul 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.41%. Comparing base (3804d73) to head (5bdbb23).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #1020   +/-   ##
========================================
  Coverage    93.41%   93.41%           
========================================
  Files           91       91           
  Lines         8667     8667           
========================================
  Hits          8096     8096           
  Misses         571      571           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3804d73...5bdbb23. Read the comment docs.

@cppalliance-bot
Copy link

@cppalliance-bot
Copy link

An automated preview of the documentation is available at https://1020.jsondocs.prtest.cppalliance.org/libs/json/doc/html/index.html

@cppalliance-bot
Copy link

@cppalliance-bot
Copy link

An automated preview of the documentation is available at https://1020.jsondocs.prtest.cppalliance.org/libs/json/doc/html/index.html

Allocations strategy is now selected separately from implementation.
All flags now support multiple choice, which allows to test several
options in one run.
@cppalliance-bot
Copy link

An automated preview of the documentation is available at https://1020.jsondocs.prtest.cppalliance.org/libs/json/doc/html/index.html

@sdarwin
Copy link
Collaborator

sdarwin commented Oct 14, 2024

bench/lib/rapidjson/include/rapidjson/document.h:2741:22: note: candidate template ignored: invalid explicitly-specified argument for template parameter 'parseFlags'
    GenericDocument& Parse(const Ch* str, size_t length) {
                     ^
bench/lib/rapidjson/include/rapidjson/document.h:2709:22: note: candidate function template not viable: requires single argument 'str', but 2 arguments were provided
    GenericDocument& Parse(const typename SourceEncoding::Ch* str) {
                     ^
bench/lib/rapidjson/include/rapidjson/document.h:2720:22: note: candidate function template not viable: requires single argument 'str', but 2 arguments were provided
    GenericDocument& Parse(const Ch* str) {
                     ^
12 errors generated.

    "clang++"   -std=c++20 -fvisibility-inlines-hidden -pthread -O3 -Wall -fvisibility=hidden -Wno-inline -m64 --target=x86_64-pc-linux  -DBOOST_ALL_NO_LIB=1 -DBOOST_COBALT_USE_STD_PMR=1 -DBOOST_CONTAINER_NO_LIB=1 -DBOOST_CONTAINER_STATIC_LINK=1 -DBOOST_JSON_HAS_NLOHMANN_JSON -DBOOST_JSON_HAS_RAPIDJSON -DBOOST_JSON_NO_LIB=1 -DBOOST_JSON_STATIC_LINK=1 -DBOOST_SYSTEM_NO_LIB=1 -DBOOST_SYSTEM_STATIC_LINK=1 -DNDEBUG   -I"../.." -I"/opt/github/boost/libs/assert/include" -I"/opt/github/boost/libs/container/include" -I"/opt/github/boost/libs/container_hash/include" -I"/opt/github/boost/libs/core/include" -I"/opt/github/boost/libs/describe/include" -I"/opt/github/boost/libs/endian/include" -I"/opt/github/boost/libs/intrusive/include" -I"/opt/github/boost/libs/move/include" -I"/opt/github/boost/libs/mp11/include" -I"/opt/github/boost/libs/predef/include" -I"/opt/github/boost/libs/system/include" -I"/opt/github/boost/libs/throw_exception/include" -I"/opt/github/boost/libs/variant2/include" -I"/opt/github/boost/libs/winapi/include" -I"include" -I"test"  -c -o "../../bin.v2/libs/json/bench/clang-linux-14/release/x86_64/cxxstd-20-iso/link-static/threading-multi/visibility-hidden/bench.o" "bench/bench.cpp"

...failed clang-linux.compile.c++ ../../bin.v2/libs/json/bench/clang-linux-14/release/x86_64/cxxstd-20-iso/link-static/threading-multi/visibility-hidden/bench.o...
clang-linux.compile.c++ ../../bin.v2/libs/json/build/clang-linux-14/release/x86_64/cxxstd-20-iso/link-static/threading-multi/visibility-hidden/src.o
clang-linux.archive ../../bin.v2/libs/json/build/clang-linux-14/release/x86_64/cxxstd-20-iso/link-static/threading-multi/visibility-hidden/libboost_json.a
...skipped <p../../bin.v2/libs/json/bench/clang-linux-14/release/x86_64/cxxstd-20-iso/link-static/threading-multi/visibility-hidden>bench for lack of <p../../bin.v2/libs/json/bench/clang-linux-14/release/x86_64/cxxstd-20-iso/link-static/threading-multi/visibility-hidden>bench.o...
...skipped <pbench>bench for lack of <p../../bin.v2/libs/json/bench/clang-linux-14/release/x86_64/cxxstd-20-iso/link-static/threading-multi/visibility-hidden>bench...

...updated 23 targets...

...skipped 2 targets...
   <p../../bin.v2/libs/json/bench/clang-linux-14/release/x86_64/cxxstd-20-iso/link-static/threading-multi/visibility-hidden>bench
   <pbench>bench

...failed updating 1 target...

@cppalliance-bot
Copy link

An automated preview of the documentation is available at https://1020.jsondocs.prtest.cppalliance.org/libs/json/doc/html/index.html

@cppalliance-bot
Copy link

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.

3 participants