================
@@ -2813,6 +2813,11 @@ void CodeGenFunction::EmitKCFIOperandBundle(
       Callee.getAbstractInfo().getCalleeFunctionProtoType();
   if (FP)
     Bundles.emplace_back("kcfi", CGM.CreateKCFITypeId(FP->desugar()));
+  else {
+    ASTContext &context = this->getContext();
+    QualType voidType = context.VoidTy;
----------------
samitolvanen wrote:

`VoidTy` doesn't seem like an appropriate type for a function even if it 
doesn't have a parameter type list. We still want to distinguish between 
different return types, right?

Also, please add a matching test when you make a change. You could probably add 
a test case for this to `clang/test/CodeGen/kcfi.c`.

https://github.com/llvm/llvm-project/pull/106677
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to