<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></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 Jan 3, 2018, at 20:47, Galina Kistanova via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" class="">llvm-commits@lists.llvm.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Hello Matt,<br class=""><br class="">It looks like this commit broke tests to one of our builders:<br class=""><a href="http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win" class="">http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win</a><br class=""><br class="">. . .<br class="">Failing Tests (1):<br class=""> LLVM :: Transforms/StructurizeCFG/AMDGPU/backedge-id-bug.ll<br class=""><br class="">Please have a look?<br class=""><br class="">Thanks<br class=""><br class="">Galina<br class=""></div><div class="gmail_extra"><br class=""><div class="gmail_quote"><br class=""></div></div></div></blockquote><br class=""></div><div>It looks like the second testcase variant has an issue with the RegionInfo construction that occurs with expensive checks / -verify-region-info enabled. It happens without the patch too, so I’ll just split that into a separate XFAILed testcase for now</div><div><br class=""></div><div>-Matt</div><br class=""></body></html>