mirror of
https://github.com/simdjson/simdjson
synced 2026-06-08 17:27:07 +00:00
strager
5510089d45
* Rename simdjson_really_inline -> simdjson_inline I want to change the simdjson_really_inline macro to sometimes not force inlining. After that upcoming change, the name simdjson_really_inline will no longer makes sense. Rename simdjson_really_inline to simdjson_inline. This patch should not change semantics; simdjson_inline still forces inlining as before. Some functions still need to be really inlined for ABI reasons. (GCC's -Wpsabi complains otherwise.) Leave those functions marked as simdjson_really_inline. * Improve build times for debug builds simdjson_inline is used for most simdjson functions. It forces inlining. In unoptimized/debug builds, this can lead to a lot of machine code being generated (especially with Address Sanitizer), causing slow compilation. Change simdjson_inline to force inlining only for optimized builds. Sometimes, the programmer might want a slightly-optimized build and want fast compilation (e.g. GCC's -Og mode). Allow simdjson users to define the simdjson_inline macro themselves (e.g. on the command line: -Dsimdjson_inline=inline) in cases where the default behavior is undesired. This patch reduced build times by over 75% for ondemand_object_tests.cpp with GCC 9.4.0 and CMAKE_BUILD_TYPE=Debug on my AMD 5950X: Before: 6.885 6.683 6.971 6.957 6.949 seconds (5 samples) After: 1.492 1.551 1.494 1.490 1.531 seconds (5 samples)
53 lines
1.3 KiB
C++
53 lines
1.3 KiB
C++
#pragma once
|
|
|
|
#if SIMDJSON_EXCEPTIONS
|
|
|
|
#include "largerandom.h"
|
|
|
|
namespace largerandom {
|
|
|
|
using namespace simdjson;
|
|
|
|
class Iter {
|
|
public:
|
|
simdjson_inline bool Run(const padded_string &json);
|
|
|
|
simdjson_inline const std::vector<my_point> &Result() { return container; }
|
|
simdjson_inline size_t ItemCount() { return container.size(); }
|
|
|
|
private:
|
|
ondemand::parser parser{};
|
|
std::vector<my_point> container{};
|
|
|
|
simdjson_inline double first_double(ondemand::json_iterator &iter) {
|
|
if (iter.start_object().error() || iter.field_key().error() || iter.field_value()) { throw "Invalid field"; }
|
|
return iter.consume_double();
|
|
}
|
|
|
|
simdjson_inline double next_double(ondemand::json_iterator &iter) {
|
|
if (!iter.has_next_field() || iter.field_key().error() || iter.field_value()) { throw "Invalid field"; }
|
|
return iter.consume_double();
|
|
}
|
|
|
|
};
|
|
|
|
simdjson_inline bool Iter::Run(const padded_string &json) {
|
|
container.clear();
|
|
|
|
auto iter = parser.iterate_raw(json).value();
|
|
if (iter.start_array()) {
|
|
do {
|
|
container.emplace_back(my_point{first_double(iter), next_double(iter), next_double(iter)});
|
|
if (iter.has_next_field()) { throw "Too many fields"; }
|
|
} while (iter.has_next_element());
|
|
}
|
|
|
|
return true;
|
|
}
|
|
|
|
BENCHMARK_TEMPLATE(LargeRandom, Iter);
|
|
|
|
} // namespace largerandom
|
|
|
|
#endif // SIMDJSON_EXCEPTIONS
|