Sync from internal Google development.

Many improvements, too many to mention.  One significant
perf regression warrants investigation:

omitfp.parsetoproto2_googlemessage1.upb_jit: 343 -> 252 (-26.53)
plain.parsetoproto2_googlemessage1.upb_jit: 334 -> 251 (-24.85)

25% regression for this benchmark is bad, but since I don't think
there's any fundamental design issue that caused it I'm going to
go ahead with the commit anyway.  Can investigate and fix later.

Other benchmarks were neutral or showed slight improvement.

1 files changed, 6 insertions, 5 deletions

diff --git a/tests/test_cpp.cc b/tests/test_cpp.cc
index 5182217..4d70e85 100644
--- a/tests/test_cpp.cc
+++ b/tests/test_cpp.cc
@@ -15,6 +15,7 @@
 #include "upb/upb.hpp"
 #include "upb/pb/decoder.hpp"
 #include "upb/pb/glue.hpp"
+#include "upb_test.h"
 
 static void TestSymbolTable(const char *descriptor_file) {
   upb::SymbolTable *s = upb::SymbolTable::New();
@@ -23,20 +24,20 @@ static void TestSymbolTable(const char *descriptor_file) {
     std::cerr << "Couldn't load descriptor: " << status;
     exit(1);
   }
-  const upb::MessageDef *md = s->LookupMessage("A");
-  assert(md);
+  const upb::MessageDef *md = s->LookupMessage("A", &md);
+  ASSERT(md);
 
   s->Unref();
-  md->Unref();
+  md->Unref(&md);
 }
 
 static void TestByteStream() {
   upb::StringSource stringsrc;
   stringsrc.Reset("testing", 7);
   upb::ByteRegion* byteregion = stringsrc.AllBytes();
-  assert(byteregion->FetchAll() == UPB_BYTE_OK);
+  ASSERT(byteregion->FetchAll() == UPB_BYTE_OK);
   char* str = byteregion->StrDup();
-  assert(strcmp(str, "testing") == 0);
+  ASSERT(strcmp(str, "testing") == 0);
   free(str);
 }