From 6e308b7e805e1baf0a63a76546f902ce0bc81501 Mon Sep 17 00:00:00 2001 From: Joshua Haberman Date: Tue, 8 Feb 2011 14:28:41 -0800 Subject: [PATCH] Fix upb's parsetostruct benchmark. --- Makefile | 4 +- benchmarks/parsetostruct.upb_table.c | 78 +++++++++++++--------------- core/upb_msg.c | 7 ++- 3 files changed, 43 insertions(+), 46 deletions(-) diff --git a/Makefile b/Makefile index 18c33d4547..a8c47cbf18 100644 --- a/Makefile +++ b/Makefile @@ -180,7 +180,9 @@ tools/upbc: core/libupb.a #UPB_BENCHMARKS=benchmarks/b.parsetostruct_googlemessage1.upb_table \ # benchmarks/b.parsetostruct_googlemessage2.upb_table UPB_BENCHMARKS=benchmarks/b.parsestream_googlemessage1.upb_table \ - benchmarks/b.parsestream_googlemessage2.upb_table + benchmarks/b.parsestream_googlemessage2.upb_table \ + benchmarks/b.parsetostruct_googlemessage1.upb_table_byref \ + benchmarks/b.parsetostruct_googlemessage2.upb_table_byref BENCHMARKS=$(UPB_BENCHMARKS) \ benchmarks/b.parsetostruct_googlemessage1.proto2_table \ diff --git a/benchmarks/parsetostruct.upb_table.c b/benchmarks/parsetostruct.upb_table.c index 494d5b711d..dfdad3657d 100644 --- a/benchmarks/parsetostruct.upb_table.c +++ b/benchmarks/parsetostruct.upb_table.c @@ -1,80 +1,76 @@ #include "main.c" -#include "upb_data.h" #include "upb_def.h" #include "upb_decoder.h" +#include "upb_strstream.h" +#include "upb_glue.h" +#include "upb_msg.h" -static upb_symtab *s; -static upb_strptr str; +static upb_string *input_str; static upb_msgdef *def; -static upb_msg *msgs[NUM_MESSAGES]; -static upb_decoder *decoder; -static upb_msgsink *sink; +static upb_msg *msg; static bool initialize() { // Initialize upb state, decode descriptor. upb_status status = UPB_STATUS_INIT; - s = upb_symtab_new(); - upb_strptr fds = upb_strreadfile(MESSAGE_DESCRIPTOR_FILE); - if(upb_string_isnull(fds)) { - fprintf(stderr, "Couldn't read " MESSAGE_DESCRIPTOR_FILE ": %s.\n", - status.msg); + upb_symtab *s = upb_symtab_new(); + upb_symtab_add_descriptorproto(s); + + upb_string *fds_str = upb_strreadfile(MESSAGE_DESCRIPTOR_FILE); + if(fds_str == NULL) { + fprintf(stderr, "Couldn't read " MESSAGE_DESCRIPTOR_FILE ":"), + upb_printerr(&status); return false; } - upb_symtab_add_desc(s, fds, &status); + upb_parsedesc(s, fds_str, &status); + upb_string_unref(fds_str); + if(!upb_ok(&status)) { - fprintf(stderr, "Error importing " MESSAGE_DESCRIPTOR_FILE ": %s.\n", - status.msg); + fprintf(stderr, "Error importing " MESSAGE_DESCRIPTOR_FILE ":"); + upb_printerr(&status); return false; } - upb_string_unref(fds); - upb_strptr proto_name = upb_strdupc(MESSAGE_NAME); - def = upb_downcast_msgdef(upb_symtab_lookup(s, proto_name)); + def = upb_dyncast_msgdef(upb_symtab_lookup(s, UPB_STRLIT(MESSAGE_NAME))); if(!def) { fprintf(stderr, "Error finding symbol '" UPB_STRFMT "'.\n", - UPB_STRARG(proto_name)); + UPB_STRARG(UPB_STRLIT(MESSAGE_NAME))); return false; } - upb_string_unref(proto_name); - - for(int i = 0; i < NUM_MESSAGES; i++) - msgs[i] = upb_msg_new(def); + upb_symtab_unref(s); // Read the message data itself. - str = upb_strreadfile(MESSAGE_FILE); - if(upb_string_isnull(str)) { + input_str = upb_strreadfile(MESSAGE_FILE); + if(input_str == NULL) { fprintf(stderr, "Error reading " MESSAGE_FILE "\n"); return false; } - decoder = upb_decoder_new(def); - sink = upb_msgsink_new(def); + upb_status_uninit(&status); + msg = upb_msg_new(def); return true; } static void cleanup() { - for(int i = 0; i < NUM_MESSAGES; i++) - upb_msg_unref(msgs[i], def); - upb_string_unref(str); - upb_symtab_unref(s); - upb_decoder_free(decoder); - upb_msgsink_free(sink); + upb_string_unref(input_str); + upb_msg_unref(msg, def); + upb_def_unref(UPB_UPCAST(def)); } static size_t run(int i) { + (void)i; upb_status status = UPB_STATUS_INIT; - upb_msg *msg = msgs[i%NUM_MESSAGES]; - upb_msgsink_reset(sink, msg); - upb_decoder_reset(decoder, upb_msgsink_sink(sink)); upb_msg_clear(msg, def); - size_t decoded = upb_decoder_decode(decoder, str, &status); - if(!upb_ok(&status) || decoded != upb_strlen(str)) { - fprintf(stderr, "Decode error: %s\n", status.msg); - return 0; - } - return upb_strlen(str); + upb_strtomsg(input_str, msg, def, &status); + if(!upb_ok(&status)) goto err; + upb_status_uninit(&status); + return upb_string_len(input_str); + +err: + fprintf(stderr, "Decode error: "); + upb_printerr(&status); + return 0; } diff --git a/core/upb_msg.c b/core/upb_msg.c index f628e3cb29..05ee1e952c 100644 --- a/core/upb_msg.c +++ b/core/upb_msg.c @@ -73,7 +73,7 @@ upb_array *upb_array_new(void) { void upb_array_recycle(upb_array **_arr, upb_fielddef *f) { upb_array *arr = *_arr; - if(arr && upb_atomic_read(&arr->refcount) == 1) { + if(arr && upb_atomic_only(&arr->refcount)) { arr->len = 0; } else { upb_array_unref(arr, f); @@ -133,7 +133,7 @@ void _upb_msg_free(upb_msg *msg, upb_msgdef *md) { void upb_msg_recycle(upb_msg **_msg, upb_msgdef *msgdef) { upb_msg *msg = *_msg; - if(msg && upb_atomic_read(&msg->refcount) == 1) { + if(msg && upb_atomic_only(&msg->refcount)) { upb_msg_clear(msg, msgdef); } else { upb_msg_unref(msg, msgdef); @@ -168,7 +168,7 @@ static void upb_msg_appendval(upb_msg *msg, upb_fielddef *f, upb_value val) { // We do: // - upb_string_recycle(), upb_string_substr() instead of // - upb_string_unref(), upb_string_getref() - // because we can conveniently caching these upb_string objects in the + // because we can conveniently cache these upb_string objects in the // upb_msg, whereas the upb_src who is sending us these strings may not // have a good way of caching them. This saves the upb_src from allocating // new upb_strings all the time to give us. @@ -190,7 +190,6 @@ upb_msg *upb_msg_appendmsg(upb_msg *msg, upb_fielddef *f, upb_msgdef *msgdef) { upb_valueptr p = upb_msg_getappendptr(msg, f); if (upb_isarray(f) || !upb_msg_has(msg, f)) { upb_msg_recycle(p.msg, msgdef); - upb_msg_clear(*p.msg, msgdef); upb_msg_sethas(msg, f); } return *p.msg;