<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 16, 2020 at 10:10 AM David Greene via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">"Doerfert, Johannes via llvm-dev" <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> writes:<br>
<br>
> The first was this one <a href="https://reviews.llvm.org/D69785" rel="noreferrer" target="_blank">https://reviews.llvm.org/D69785</a>, since then the<br>
> series grew in all directions (see the stack).<br>
<br>
How do I see the stack?<br>
<br>
This is a great example of the (to me) unintuitive interface of Phab.<br>
People talk about things like "the stack" as if it's obvious and it's<br>
really not.<br></blockquote><div><br>Yeah, once something's got a lot of commenting on it - but there's two instances of the word "stack" when I open and search that page. The first one is the one you're looking for I think - under "Revision Contents" there's "Files", "History", "Commits", "Stack", and "Similar".<br><br>If you open "Stack" it shows various related and not so related commits in the timeline - if you look at the line related to this review (highlighted in yellow) you can see that this patch is on the purple coloured stack. So you can see patches built up on top of it, such as D69922.<br><br>If you're reviewing this patch - in the timeline of comments you'd see a structured event description like "<a href="https://reviews.llvm.org/p/jdoerfert/" class="gmail-phui-handle gmail-phui-link-person" style="color:rgb(75,77,81);font-weight:bold;white-space:nowrap;font-family:"Segoe UI","Segoe UI Emoji","Segoe UI Symbol",Lato,"Helvetica Neue",Helvetica,Arial,sans-serif;font-size:13px;background-color:rgb(243,245,247)">jdoerfert</a><span style="color:rgb(116,119,125);font-family:"Segoe UI","Segoe UI Emoji","Segoe UI Symbol",Lato,"Helvetica Neue",Helvetica,Arial,sans-serif;font-size:13px;background-color:rgb(243,245,247)"> added a child revision: </span><a href="https://reviews.llvm.org/D69922" class="gmail-phui-handle gmail-handle-status-closed" style="text-decoration-line:line-through;color:rgb(75,77,81);font-weight:bold;font-family:"Segoe UI","Segoe UI Emoji","Segoe UI Symbol",Lato,"Helvetica Neue",Helvetica,Arial,sans-serif;font-size:13px;background-color:rgb(243,245,247)">D69922: [OpenMP] Use the OpenMP-IR-Builder</a><span style="color:rgb(116,119,125);font-family:"Segoe UI","Segoe UI Emoji","Segoe UI Symbol",Lato,"Helvetica Neue",Helvetica,Arial,sans-serif;font-size:13px;background-color:rgb(243,245,247)">." telling you that there was a new patch added that is based on top of this one.<br></span><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
                        -David<br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
</blockquote></div></div>