<div dir="ltr">Hi Chuck, can you resubmit this patch attached to the other review?  The workflow isn't entirely obvious, but there's a couple of things you need to do different.<br><div><br></div><div>1) When you create the diff with the suggested changes, create the diff against origin (or whatever you created the *original* patch against), not against the first version of the patch.  So if HEAD is the first version of the patch you uploaded, create your diff against HEAD~1.  You want the new patch to contain the changes _plus_ the original patch.</div><div><br></div><div>2) When you attach the patch to Phabricator, you'll get a combo box and the default value is "Create new revision".  Change this to "Attach to existing revision" and then choose the entry for the first patch you uploaded.</div></div><br><div class="gmail_quote">On Tue, Mar 10, 2015 at 4:49 PM Chuck Ries <<a href="mailto:chuckr@microsoft.com">chuckr@microsoft.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi abidh,<br>
<br>
<a href="http://reviews.llvm.org/D8238" target="_blank">http://reviews.llvm.org/D8238</a><br>
<br>
Files:<br>
  tools/lldb-mi/<u></u>MICmnLLDBDebugSessionInfo.cpp<br>
  tools/lldb-mi/<u></u>MICmnLLDBDebugSessionInfo.h<br>
<br>
Index: tools/lldb-mi/<u></u>MICmnLLDBDebugSessionInfo.cpp<br>
==============================<u></u>==============================<u></u>=======<br>
--- tools/lldb-mi/<u></u>MICmnLLDBDebugSessionInfo.cpp<br>
+++ tools/lldb-mi/<u></u>MICmnLLDBDebugSessionInfo.cpp<br>
@@ -1048,7 +1048,7 @@<br>
 bool<br>
 CMICmnLLDBDebugSessionInfo::<u></u>MIResponseFormBrkPtFrameInfo(<u></u>const SBrkPtInfo &vrBrkPtInfo, CMICmnMIValueTuple &vwrMiValueTuple)<br>
 {<br>
-    const CMIUtilString strAddr(CMIUtilString::Format(<u></u>"0x%08llx", vrBrkPtInfo.m_pc));<br>
+    const CMIUtilString strAddr(CMIUtilString::Format(<u></u>"0x%016llx", vrBrkPtInfo.m_pc));<br>
     const CMICmnMIValueConst miValueConst2(strAddr);<br>
     const CMICmnMIValueResult miValueResult2("addr", miValueConst2);<br>
     if (!vwrMiValueTuple.Add(<u></u>miValueResult2))<br>
Index: tools/lldb-mi/<u></u>MICmnLLDBDebugSessionInfo.h<br>
==============================<u></u>==============================<u></u>=======<br>
--- tools/lldb-mi/<u></u>MICmnLLDBDebugSessionInfo.h<br>
+++ tools/lldb-mi/<u></u>MICmnLLDBDebugSessionInfo.h<br>
@@ -87,7 +87,7 @@<br>
         CMIUtilString m_strType;        // Break point type.<br>
         bool m_bDisp;                   // True = "del", false = "keep".<br>
         bool m_bEnabled;                // True = enabled, false = disabled break point.<br>
-        MIuint m_pc;                    // Address number.<br>
+        MIuint64 m_pc;                  // Address number.<br>
         CMIUtilString m_fnName;         // Function name.<br>
         CMIUtilString m_fileName;       // File name text.<br>
         CMIUtilString m_path;           // Full file name and path text.<br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
______________________________<u></u>_________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@cs.uiuc.edu" target="_blank">lldb-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/lldb-commits</a><br>
</blockquote></div>