]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/compile: mention issue 28603 when discussing where inline marks go
authorKeith Randall <keithr@alum.mit.edu>
Thu, 16 May 2019 23:16:26 +0000 (16:16 -0700)
committerKeith Randall <khr@golang.org>
Fri, 17 May 2019 03:46:38 +0000 (03:46 +0000)
Update #28603

Change-Id: I8bdbea57b3661e15dcc760d27252b794daa476db
Reviewed-on: https://go-review.googlesource.com/c/go/+/177697
Reviewed-by: Emmanuel Odeke <emm.odeke@gmail.com>
src/cmd/compile/internal/gc/inl.go

index addba98e3ec04b88dfe0c189ed580082af9396eb..b41b8cb1a490bb3debce6c0619ea433384707593 100644 (file)
@@ -1046,10 +1046,11 @@ func mkinlcall(n, fn *Node, maxCost int32) *Node {
        }
        newIndex := Ctxt.InlTree.Add(parent, n.Pos, fn.Sym.Linksym())
 
-       // Add a inline mark just before the inlined body.
+       // Add an inline mark just before the inlined body.
        // This mark is inline in the code so that it's a reasonable spot
        // to put a breakpoint. Not sure if that's really necessary or not
        // (in which case it could go at the end of the function instead).
+       // Note issue 28603.
        inlMark := nod(OINLMARK, nil, nil)
        inlMark.Pos = n.Pos.WithIsStmt()
        inlMark.Xoffset = int64(newIndex)