[compiler] Remove need for passing Module to NameMangler

This was only used for its LLVMContext, which we already provide to the
mangler upon construction - perhaps for legacy reasons.

Author: frasercrmck

llvm/lib/SYCLNativeCPUUtils/compiler_passes/compiler_pipeline/include/multi_llvm/multi_llvm.h

@@ -63,9 +63,9 @@ inline llvm::InlineResult InlineFunction(llvm::CallInst *CI,
 #endif
 }
 
-inline llvm::StructType *getStructTypeByName(llvm::Module &module,
+inline llvm::StructType *getStructTypeByName(llvm::LLVMContext &ctx,
                                              llvm::StringRef name) {
-  return llvm::StructType::getTypeByName(module.getContext(), name);
+  return llvm::StructType::getTypeByName(ctx, name);
 }
 
 inline llvm::DILocation *getDILocation(unsigned Line, unsigned Column,

llvm/lib/SYCLNativeCPUUtils/compiler_passes/vecz/source/vectorization_context.cpp

@@ -143,7 +143,7 @@ VectorizationResult &VectorizationContext::getOrCreateBuiltin(
     return result;
   }
 
-  compiler::utils::NameMangler Mangler(&F.getContext(), &Module);
+  compiler::utils::NameMangler Mangler(&F.getContext());
   auto const BuiltinName = Mangler.demangleName(F.getName()).str();
 
   result.func = VectorCallee;