diff --git a/upb/test_table.cc b/upb/test_table.cc
index e1a1d40733..2c368f839c 100644
--- a/upb/test_table.cc
+++ b/upb/test_table.cc
@@ -354,7 +354,6 @@ class TypedIntTable {
 
 }  // namespace upb
 
-bool benchmark = false;
 #define CPU_TIME_PER_TEST 0.5
 
 using std::vector;
diff --git a/upbc/protoc-gen-upb.cc b/upbc/protoc-gen-upb.cc
index 4703fa7033..da19286766 100644
--- a/upbc/protoc-gen-upb.cc
+++ b/upbc/protoc-gen-upb.cc
@@ -312,7 +312,7 @@ bool HasNonZeroDefault(const protobuf::FieldDescriptor* field) {
       return field->default_value_enum()->number() != 0;
   }
   ABSL_ASSERT(false);
-  return "XXX";
+  return false;
 }
 
 std::string FieldDefault(const protobuf::FieldDescriptor* field) {