<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Dec 13, 2016 at 4:57 PM, Grang, Mandeep Singh via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    <p>Currently, the following unit tests are failing in
      Release+Asserts mode due to the difference in SmallPtrSet
      iteration order:<br></p><p><span class="">
          Clang :: Analysis/keychainAPI.m<br>
          Clang :: Analysis/malloc.c<br>
          Clang :: Rewriter/<a href="http://objc-modern-metadata-visibility.mm" target="_blank">objc-modern-metadata-<wbr>visibility.mm</a><br>
          Clang :: SemaCXX/warn-loop-analysis.cpp<br></span>
          LLVM :: Transforms/SimplifyCFG/bug-<wbr>25299.ll</p></div></blockquote><div>Just to be clear, committing this intentionally with the knowledge that you were breaking tests in LLVM and Clang without a plan to fix them in a timely fashion was completely unacceptable. There are a lot of people who work really hard to keep our buildbots green, and committing patches that break them creates work for them and blinds us to issues that crop up while the bots are red.</div><div><br></div><div>It sounds like everything is currently on the right track, though, so I'm looking forward to seeing this work finished.</div></div></div></div>