Comment 1:

Here's what needs to be done to goapi to make it faster, once this is fixed:
diff -r 821585f8baba src/cmd/api/goapi.go
--- a/src/cmd/api/goapi.go      Fri Nov 16 11:53:26 2012 -0800
+++ b/src/cmd/api/goapi.go      Fri Nov 16 13:15:51 2012 -0800
@@ -353,6 +353,15 @@
        return
 }
 
+var parsedFileCache = make(map[string]*ast.File)
+
+func parseFile(filename string) (*ast.File, error) {
+       if f, ok := parsedFileCache[filename]; ok {
+               return f, nil
+       }
+       return parser.ParseFile(fset, filename, nil, 0)
+}
+
 // WalkPackage walks all files in package `name'.
 // WalkPackage does nothing if the package has already been loaded.
 func (w *Walker) WalkPackage(name string) {
@@ -386,7 +395,8 @@
 
        files := append(append([]string{}, info.GoFiles...), info.CgoFiles...)
        for _, file := range files {
-               f, err := parser.ParseFile(fset, filepath.Join(dir, file), nil, 0)
+               filename := filepath.Join(dir, file)
+               f, err := parseFile(filename)
                if err != nil {
                        log.Fatalf("error parsing package %s, file %s: %v", name, file, err)
                }
@@ -395,6 +405,8 @@
                for _, dep := range fileDeps(f) {
                        w.WalkPackage(dep)
                }
+               parsedFileCache[filename] = f
+
        }
 
        if *verbose {

Comment 2:

(or actually parseFile can populate the cache itself.  I tried moving it out to where it
is now when debugging the problem, before I remembered this issue...)

Comment 3:

This issue was updated by revision aeca7a7.

R=golang-dev, remyoudompheng, minux.ma, gri
CC=golang-dev
http://golang.org/cl/6845058

Comment 4:

Labels changed: added priority-later, removed priority-triage.

Comment 5:

Labels changed: removed go1.1maybe.

Comment 6:

Labels changed: added go1.3maybe.

Comment 7:

Labels changed: added release-none, removed go1.3maybe.

Comment 8:

Labels changed: added repo-main.

Comment 9:

go/doc.New is still destructive but goapi was rewritten to use go/types instead.
Closing this since it doesn't affect goapi anymore.

Status changed to Fixed.