[NFC] Add a safety assert to BuiltinCallMutator. (#1748)

jcranmer-intel · web-flow · commit cff010d4397c · 2022-11-30T01:05:50.000+01:00
diff --git a/lib/SPIRV/SPIRVBuiltinHelper.cpp b/lib/SPIRV/SPIRVBuiltinHelper.cpp
@@ -208,6 +208,7 @@ BuiltinCallMutator BuiltinCallHelper::mutateCallInst(CallInst *CI,
 
 BuiltinCallMutator BuiltinCallHelper::mutateCallInst(CallInst *CI,
                                                      std::string FuncName) {
+  assert(CI->getCalledFunction() && "Can only mutate direct function calls.");
   return BuiltinCallMutator(CI, std::move(FuncName), Rules, NameMapFn);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -208,6 +208,7 @@ BuiltinCallMutator BuiltinCallHelper::mutateCallInst(CallInst *CI,`
`208`	`208`
`209`	`209`	`BuiltinCallMutator BuiltinCallHelper::mutateCallInst(CallInst *CI,`
`210`	`210`	`std::string FuncName) {`
	`211`	`+ assert(CI->getCalledFunction() && "Can only mutate direct function calls.");`
`211`	`212`	`return BuiltinCallMutator(CI, std::move(FuncName), Rules, NameMapFn);`
`212`	`213`	`}`
`213`	`214`