reviewer feedback, comment changes

Author: cperkinsintel

clang/tools/clang-offload-wrapper/ClangOffloadWrapper.cpp

@@ -1303,7 +1303,7 @@ class BinaryWrapper {
 
     // Declaration for __sycl_unregister_lib(void*).
     auto *UnregTargetTy =
-        FunctionType::get(Type::getVoidTy(C), getPtrTy(), false);
+        FunctionType::get(Type::getVoidTy(C), getPtrTy(), /*isVarArg=*/false);
     FunctionCallee UnregTargetC =
         M.getOrInsertFunction("__sycl_unregister_lib", UnregTargetTy);
 
@@ -1321,8 +1321,7 @@ class BinaryWrapper {
     FunctionCallee RegTargetC =
         M.getOrInsertFunction("__sycl_register_lib", RegTargetTy);
 
-    // `atexit` takes a `void(*)()` function pointer. In LLVM IR, this is
-    // typically represented as `i32 (ptr)`.
+    // `atexit` takes a `void(*)()` function pointer arg and returns an i32.
     FunctionType *AtExitTy =
         FunctionType::get(Type::getInt32Ty(C), getPtrTy(), false);
     FunctionCallee AtExitC = M.getOrInsertFunction("atexit", AtExitTy);

llvm/lib/Frontend/Offloading/SYCLOffloadWrapper.cpp

@@ -765,8 +765,7 @@ struct Wrapper {
     FunctionCallee RegTargetC =
         M.getOrInsertFunction("__sycl_register_lib", RegTargetTy);
 
-    // `atexit` takes a `void(*)()` function pointer. In LLVM IR, this is
-    // typically represented as `i32 (ptr)`.
+    // `atexit` takes a `void(*)()` function pointer arg and returns an i32.
     FunctionType *AtExitTy = FunctionType::get(
         Type::getInt32Ty(C), PointerType::getUnqual(C), false);
     FunctionCallee AtExitC = M.getOrInsertFunction("atexit", AtExitTy);