<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On Dec 3, 2020, at 5:43 PM, Robinson, Paul <<a href="mailto:paul.robinson@sony.com" class="">paul.robinson@sony.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><blockquote type="cite" style="font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none;" class="">-----Original Message-----<br class="">From: Jim Ingham <<a href="mailto:jingham@apple.com" class="">jingham@apple.com</a>><br class="">Sent: Thursday, December 3, 2020 5:51 PM<br class="">To: David Blaikie<br class=""><<a href="mailto:reviews+D91734+public+9741d1f0c1792e0e@reviews.llvm.org" class="">reviews+D91734+public+9741d1f0c1792e0e@reviews.llvm.org</a>>; David Blaikie<br class="">via Phabricator <<a href="mailto:reviews@reviews.llvm.org" class="">reviews@reviews.llvm.org</a>><br class="">Cc: Robinson, Paul <<a href="mailto:paul.robinson@sony.com" class="">paul.robinson@sony.com</a>>;<span class="Apple-converted-space"> </span><a href="mailto:rnk@google.com" class="">rnk@google.com</a>;<br class=""><a href="mailto:echristo@gmail.com" class="">echristo@gmail.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:sontuan.vu119@gmail.com" class="">sontuan.vu119@gmail.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:mcrosier@codeaurora.org" class="">mcrosier@codeaurora.org</a>;<br class=""><a href="mailto:nikola.tesic@syrmia.com" class="">nikola.tesic@syrmia.com</a>; lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org" class="">lldb-commits@lists.llvm.org</a>>;<br class=""><a href="mailto:Liburd1994@outlook.com" class="">Liburd1994@outlook.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:stefan.graenitz@gmail.com" class="">stefan.graenitz@gmail.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:matze@braunis.de" class="">matze@braunis.de</a>;<br class=""><a href="mailto:nemanja.i.ibm@gmail.com" class="">nemanja.i.ibm@gmail.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:simon.dardis@gmail.com" class="">simon.dardis@gmail.com</a>; Tozer, Stephen<br class=""><<a href="mailto:stephen.tozer@sony.com" class="">stephen.tozer@sony.com</a>>;<span class="Apple-converted-space"> </span><a href="mailto:AlokKumar.Sharma@amd.com" class="">AlokKumar.Sharma@amd.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:sbc@chromium.org" class="">sbc@chromium.org</a>;<br class=""><a href="mailto:sourabhsingh.tomar@amd.com" class="">sourabhsingh.tomar@amd.com</a>; Jackson, Chris <<a href="mailto:chris.jackson@sony.com" class="">chris.jackson@sony.com</a>>;<br class=""><a href="mailto:arphaman@gmail.com" class="">arphaman@gmail.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:jji@us.ibm.com" class="">jji@us.ibm.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:88888yl@gmail.com" class="">88888yl@gmail.com</a>;<br class=""><a href="mailto:horse10241024@gmail.com" class="">horse10241024@gmail.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:pengfei.wang@intel.com" class="">pengfei.wang@intel.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:david.green@arm.com" class="">david.green@arm.com</a>;<br class=""><a href="mailto:higuoxing@gmail.com" class="">higuoxing@gmail.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:ikudrin@accesssoftek.com" class="">ikudrin@accesssoftek.com</a>; p8u8i7l5t1q9r8w3@ibm-<br class=""><a href="http://systems-z.slack.com/" class="">systems-z.slack.com</a>; Enciso, Carlos <<a href="mailto:Carlos.Enciso@sony.com" class="">Carlos.Enciso@sony.com</a>>; llvm-<br class=""><a href="mailto:commits@lists.llvm.org" class="">commits@lists.llvm.org</a>; Cazalet-Hyams, Orlando <<a href="mailto:orlando.hyams@sony.com" class="">orlando.hyams@sony.com</a>>;<br class=""><a href="mailto:asowda@cisco.com" class="">asowda@cisco.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:sanimir@subpath.org" class="">sanimir@subpath.org</a>;<span class="Apple-converted-space"> </span><a href="mailto:simon@atanasyan.com" class="">simon@atanasyan.com</a>;<br class=""><a href="mailto:jrtc27@jrtc27.com" class="">jrtc27@jrtc27.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:djordje.todorovic@syrmia.com" class="">djordje.todorovic@syrmia.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:ch.bessonova@gmail.com" class="">ch.bessonova@gmail.com</a>;<br class=""><a href="mailto:quentin.colombet@gmail.com" class="">quentin.colombet@gmail.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:akhuang@google.com" class="">akhuang@google.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:aheejin@gmail.com" class="">aheejin@gmail.com</a>;<br class=""><a href="mailto:avl.lapshin@gmail.com" class="">avl.lapshin@gmail.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:david.stenberg@ericsson.com" class="">david.stenberg@ericsson.com</a>;<br class=""><a href="mailto:david.spickett@linaro.org" class="">david.spickett@linaro.org</a>;<span class="Apple-converted-space"> </span><a href="mailto:dougpuob@gmail.com" class="">dougpuob@gmail.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:bhuvanendra.kumarn@amd.com" class="">bhuvanendra.kumarn@amd.com</a>;<br class=""><a href="mailto:serhiy.redko@gmail.com" class="">serhiy.redko@gmail.com</a>;<span class="Apple-converted-space"> </span><a href="mailto:jini.susan.george@gmail.com" class="">jini.susan.george@gmail.com</a><br class="">Subject: Re: [Lldb-commits] [PATCH] D91734: [FastISel] Flush local value<br class="">map on every instruction<br class=""><br class="">Note, lldb has a bunch of special handling for line 0 code. For instance,<br class="">when we are pushing a breakpoint past the prologue we will keep pushing it<br class="">forward over line number 0 lines. Those are compiler generated and in<br class="">general people don't want to stop there. Similarly, if you are stepping<br class="">through line 3 and the next line entry after 3 is line 0 we keep stepping<br class="">till we get to a non-zero line.<br class=""><br class="">When the compiler is actually using line 0 to mean "compiler generated<br class="">code not really associated with a particular line, then I am pretty sure<br class="">the debugger has to be aware of this or debugging is going to be a bit<br class="">awkward...<br class=""><br class="">I don't know if that's directly relevant to this bug, I haven't had time<br class="">to follow the whole discussion. But I'm not convinced all the problems<br class="">with line 0 emission causing debugging oddities can be solved in the line<br class="">table generation.<br class=""><br class="">Jim<br class=""></blockquote><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">Hi Jim,</span><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">This issue is not really "line 0 causing debugger oddities" so much as</span><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">"line 0 attached to instructions that should have a real line number."</span><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">It's true that gdb's response to line 0 on the first instruction past</span><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">the prologue_end seems a tad idiosyncratic, but if nothing else, it</span><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">exposed an issue in the compiler that is worth solving. There are</span><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">clearly instructions attributed to line 0 that shouldn't be, and the</span><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">goal here is to make that better.</span><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""></div></blockquote><div><br class=""></div><div>That's great! Getting all the instructions you can assigned correctly to real lines is worthy labor! </div><div><br class=""></div><div>Once you are all done with that work, given you really are using line 0 for real instances of compiler generated code that aren't attributable to any one line, stepping in gdb is going to look odd sometimes if gdb doesn't know to handle them. It might be worth filing some bugs with gdb to cope with this situation (though its probably better to use examples where the line 0 is correctly attributed...)</div><div><br class=""></div><div>Jim</div><div><br class=""></div><br class=""><blockquote type="cite" class=""><div class=""><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">Thanks,</span><br style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">--paulr</span></div></blockquote></div><br class=""></body></html>