Fix void* -> i8* conversion

Author: pmenon

src/codegen/operator/csv_scan_translator.cpp

@@ -74,8 +74,8 @@ void CSVScanTranslator::InitializeQueryState() {
   // Cast the runtime type to an opaque void*. This is because we're calling
   // into pre-compiled C++ that doesn't know that the dynamically generated
   // RuntimeState* looks like.
-  llvm::Value *query_state_ptr = codegen->CreatePointerCast(
-      codegen.GetState(), codegen.VoidType()->getPointerTo());
+  llvm::Value *query_state_ptr =
+      codegen->CreatePointerCast(codegen.GetState(), codegen.VoidPtrType());
 
   // Call CSVScanner::Init()
   codegen.Call(CSVScannerProxy::Init,

src/include/codegen/proxy/type_builder.h

@@ -53,6 +53,9 @@ DEFINE_PRIMITIVE_BUILDER(unsigned long, Int64);
 DEFINE_PRIMITIVE_BUILDER(long long, Int64);
 DEFINE_PRIMITIVE_BUILDER(unsigned long long, Int64);
 DEFINE_PRIMITIVE_BUILDER(double, Double);
+DEFINE_PRIMITIVE_BUILDER(void *, VoidPtr);
+DEFINE_PRIMITIVE_BUILDER(char *, CharPtr);
+DEFINE_PRIMITIVE_BUILDER(unsigned char *, CharPtr);
 #undef DEFINE_PRIMITIVE_BUILDER
 
 /// Const