<div dir="ltr"><div>Again, thanks for root-causing this!</div><div><br></div>Fixed the cause for the regression in r190126. However, I don't think we address this well in all cases. E.g. if getTopLevelRegion() had a parameter, we would still break after that call (and we "always" have). Please file a bug, if this comes up!</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 6, 2013 at 1:20 AM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 08/27/2013 07:24 AM, Daniel Jasper wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: djasper<br>
Date: Tue Aug 27 09:24:43 2013<br>
New Revision: 189353<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=189353&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=189353&view=rev</a><br>
Log:<br>
clang-format: Format segments of builder-type calls one per line.<br>
<br>
This fixes <a href="http://llvm.org/PR14818" target="_blank">llvm.org/PR14818</a>.<br>
<br>
Before:<br>
   return llvm::StringSwitch<Reference::<u></u>Kind>(name)<br>
              .StartsWith(".eh_frame_hdr", ORDER_EH_FRAMEHDR)<br>
              .StartsWith(".eh_frame", ORDER_EH_FRAME)<br>
              .StartsWith(".init", ORDER_INIT).StartsWith(".fini"<u></u>, ORDER_FINI)<br>
              .StartsWith(".hash", ORDER_HASH).Default(ORDER_<u></u>TEXT);<br>
<br>
After:<br>
   return llvm::StringSwitch<Reference::<u></u>Kind>(name)<br>
              .StartsWith(".eh_frame_hdr", ORDER_EH_FRAMEHDR)<br>
              .StartsWith(".eh_frame", ORDER_EH_FRAME)<br>
              .StartsWith(".init", ORDER_INIT)<br>
              .StartsWith(".fini", ORDER_FINI)<br>
              .StartsWith(".hash", ORDER_HASH)<br>
              .Default(ORDER_TEXT);<br>
</blockquote>
<br></div>
This change introduced the following regression:<br>
<br>
Before:<br>
<br>
int foo() {<br>
  if (1)<br>
    if (1)<br>
      if (1)<br>
        O.indent(2 * (depth + 1))<br>
            << "Node"<br>
            << static_cast<void *>(RI->getTopLevelRegion()-><u></u>getBBNode(*BI))<br>
            << ";\n";<br>
}<br>
<br>
After:<br>
<br>
int foo() {<br>
  if (1)<br>
    if (1)<br>
      if (1)<br>
        O.indent(2 * (depth + 1)) << "Node"<br>
                                  << static_cast<void *>(RI->getTopLevelRegion()<br>
<br>
->getBBNode(*BI))<br>
                                  << ";\n";<br>
}<br>
<br>
I believe moving the << to a new line is better to read than adding a line break before the call to ->getBBNode().<br>
<br>
Cheers,<br>
Tobias<br>
</blockquote></div><br></div>