[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