Fix comments

KhronosGroup · Jul 1, 2020 · 584514c · 584514c
1 parent ed00899
commit 584514c
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/source/fuzz/protobufs/spvtoolsfuzz.proto b/source/fuzz/protobufs/spvtoolsfuzz.proto
@@ -920,7 +920,7 @@ message TransformationFunctionCall {
 message TransformationInvertComparisonOperator {
 
   // For some instruction with result id |operator_id| that
-  // represents a binary operator (<, >, <=), this transformation
+  // represents a binary comparison operator (e.g. <, >, <=), this transformation
   // will replace that instruction's result id with |fresh_id|,
   // invert the opcode (< will become >=) and insert OpLogicalNot
   // instruction with result id |operator_id| below.

diff --git a/source/fuzz/transformation_invert_comparison_operator.cpp b/source/fuzz/transformation_invert_comparison_operator.cpp
@@ -57,7 +57,7 @@ bool TransformationInvertComparisonOperator::IsApplicable(
 void TransformationInvertComparisonOperator::Apply(
     opt::IRContext* ir_context, TransformationContext* /*unused*/) const {
   auto* inst = ir_context->get_def_use_mgr()->GetDef(message_.operator_id());
-  assert(inst);
+  assert(inst && "Result id of an operator is invalid");
 
   // Insert negation after |inst|.
   auto iter = fuzzerutil::GetIteratorForInstruction(

diff --git a/source/fuzz/transformation_invert_comparison_operator.h b/source/fuzz/transformation_invert_comparison_operator.h
@@ -32,7 +32,7 @@ class TransformationInvertComparisonOperator : public Transformation {
                                          uint32_t fresh_id);
 
   // - |operator_id| should be a result id of some instruction for which
-  //   InInversionSupported returns true.
+  //   IsInversionSupported returns true.
   // - |fresh_id| must be a fresh id.
   bool IsApplicable(
       opt::IRContext* ir_context,