[PATCH] [llgo] irgen: make it possible to use a custom importer

Peter Collingbourne peter at pcc.me.uk
Tue Dec 30 16:26:52 PST 2014


================
Comment at: irgen/compiler.go:89
@@ +88,3 @@
+
+	// InitMap is the init map used by Importer. If nil, the compiler will
+	// set this field automatically using MakeImporter().
----------------
axw wrote:
> If *Importer* is nil, the compiler will set this field automatically using MakeImporter(). If Importer is non-nil, InitMap must be non-nil also (I think?)
Yes, done.

================
Comment at: irgen/compiler.go:161
@@ -153,1 +160,3 @@
 
+func (opts *CompilerOptions) MakeImporter() error {
+	opts.InitMap = make(map[*types.Package]gccgoimporter.InitData)
----------------
axw wrote:
> A brief comment may be useful here, describing the interaction with GccgoPath.
Done.

http://reviews.llvm.org/D6803

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list