<div><div dir="auto">I see. No problem then, I’ll compose a test case.</div></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 14 Mar 2020 at 0:07 Florian Hahn <<a href="mailto:florian_hahn@apple.com">florian_hahn@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space"><br><div><br><blockquote type="cite"><div>On Mar 13, 2020, at 19:03, Ehud Katz via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:</div><br><div><div dir="ltr"><a href="https://github.com/llvm/llvm-project/commit/fcc2238b8bfb9498b46a48d219193de8c00a25a0" target="_blank">rGfcc2238b8bfb</a> broke the sanitizer "annotate" tests - <a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/26137/" rel="noreferrer" style="text-decoration-line:none;color:rgb(19,108,178);margin-top:0px;font-family:"Segoe UI","Segoe UI Emoji","Segoe UI Symbol",Lato,"Helvetica Neue",Helvetica,Arial,sans-serif;font-size:13px" target="_blank">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/26137/</a>, which inherently, I think, seems like a good enough test case.<div><a href="https://github.com/llvm/llvm-project/commit/18eae3312297cb197a3131f3ad9ca2bebb217415" target="_blank">rG18eae3312297</a> fixes it.<br></div></div></div></blockquote></div><br><div>Usually the bots should not be the first line of defense, the unit tests in the repo should be. The bot might go away or some other changes in other parts of LLVM/Clang/Sanitizers may change and hide the failure.</div><div><br></div><div>If possible, please try to get a reduced IR test case for the failure. This will provide a more solid guard against the error in the future.</div><div><br></div><div>Cheers,</div><div>Florian</div></div></blockquote></div></div>