<div dir="ltr">(might be useful to include [llgo] or something similar in the subject line in the future to help reviewers filter/prioritize appropriately - took me a moment to realize this wasn't something I needed to worry about)<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 27, 2015 at 11:43 PM, Andrew Wilkins <span dir="ltr"><<a href="mailto:axwalk@gmail.com" target="_blank">axwalk@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi pcc,<br>
<br>
The debug metadata we generate is wrong, and is<br>
now causing build failures. This revision disables<br>
the only llvm.dbg.declare calls we make.<br>
<br>
(There is also a drive-by fix to CMakeLists.txt,<br>
adding in a missing .go dependency.)<br>
<br>
<a href="http://reviews.llvm.org/D7222" target="_blank">http://reviews.llvm.org/D7222</a><br>
<br>
Files:<br>
  CMakeLists.txt<br>
  irgen/ssa.go<br>
<br>
Index: CMakeLists.txt<br>
===================================================================<br>
--- CMakeLists.txt<br>
+++ CMakeLists.txt<br>
@@ -30,6 +30,7 @@<br>
   irgen/slice.go<br>
   irgen/ssa.go<br>
   irgen/strings.go<br>
+  irgen/switches.go<br>
   irgen/targets.go<br>
   irgen/typemap.go<br>
   irgen/types.go<br>
Index: irgen/ssa.go<br>
===================================================================<br>
--- irgen/ssa.go<br>
+++ irgen/ssa.go<br>
@@ -357,12 +357,7 @@<br>
        prologueBlock := llvm.InsertBasicBlock(fr.blocks[0], "prologue")<br>
        fr.builder.SetInsertPointAtEnd(prologueBlock)<br>
<br>
-       // Map parameter positions to indices. We use this<br>
-       // when processing locals to map back to parameters<br>
-       // when generating debug metadata.<br>
-       paramPos := make(map[token.Pos]int)<br>
        for i, param := range f.Params {<br>
-               paramPos[param.Pos()] = i<br>
                llparam := fti.argInfos[i].decode(llvm.GlobalContext(), fr.builder, fr.builder)<br>
                if isMethod && i == 0 {<br>
                        if _, ok := param.Type().Underlying().(*types.Pointer); !ok {<br>
@@ -401,13 +396,6 @@<br>
                bcalloca := fr.builder.CreateBitCast(alloca, llvm.PointerType(llvm.Int8Type(), 0), "")<br>
                value := newValue(bcalloca, local.Type())<br>
                fr.env[local] = value<br>
-               if fr.GenerateDebug {<br>
-                       paramIndex, ok := paramPos[local.Pos()]<br>
-                       if !ok {<br>
-                               paramIndex = -1<br>
-                       }<br>
-                       fr.debug.Declare(fr.builder, local, alloca, paramIndex)<br>
-               }<br>
        }<br>
<br>
        // If the function contains any defers, we must first create<br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div></div>