[PATCH] llgo: Generate llvm.dbg.value calls

Peter Collingbourne peter at pcc.me.uk
Sun Dec 14 18:50:30 PST 2014


This probably will not apply correctly after the changes I made in r224180. Would you mind updating and fixing any merge conflicts? Thanks.


================
Comment at: debug/debug.go:97
@@ -96,3 +96,3 @@
 	}
-	diFile := d.builder.CreateFile(d.remapFilePath(file.Name()), "")
+	diFile := d.builder.CreateFile(d.remapFilePath(file.Name()), d.cwd)
 	d.files[file] = diFile
----------------
The cwd changes seem unrelated. Can you move those to a separate revision, please?

================
Comment at: irgen/ssa.go:888
@@ +887,3 @@
+		}
+		fr.debug.Value(fr.builder, id.Name, fr.llvmvalue(instr.X), instr.X.Type(), id.Obj.Pos(), paramIndex)
+
----------------
Perhaps you want to check `IsAddr` to decide whether to emit an `llvm.dbg.value` here or not?

http://reviews.llvm.org/D6651

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






More information about the llvm-commits mailing list