<div dir="ltr">Hi Vlad,<div><br></div><div>I just submitted a fix for this: e11f9fb4508534d31b09d2ba6cd22428ccc75f65</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Feb 25, 2020 at 11:42 AM Vlad Vereschaka via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.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">vvereschaka added a comment.<br>
<br>
Hello @void ,<br>
<br>
this commit breaks Aarch64 builder with failed `Clang::uninit-asm-goto.cpp` test<br>
<a href="http://lab.llvm.org:8011/builders/llvm-clang-win-x-aarch64/builds/5020" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/llvm-clang-win-x-aarch64/builds/5020</a><br>
<br>
- FAIL: Clang::uninit-asm-goto.cpp<br>
<br>
******************** TEST 'Clang :: Analysis/uninit-asm-goto.cpp' FAILED ********************<br>
Script:<br>
--<br>
: 'RUN: at line 1'; c:\buildbot\as-builder-2\llvm-clang-win-x-aarch64\build\bin\clang.exe -cc1 -internal-isystem c:\buildbot\as-builder-2\llvm-clang-win-x-aarch64\build\lib\clang\11.0.0\include -nostdsysteminc -std=c++11 -Wuninitialized -verify C:\buildbot\as-builder-2\llvm-clang-win-x-aarch64\llvm-project\clang\test\Analysis\uninit-asm-goto.cpp<br>
--<br>
Exit Code: 1<br>
<br>
Command Output (stdout):<br>
--<br>
$ ":" "RUN: at line 1"<br>
$ "c:\buildbot\as-builder-2\llvm-clang-win-x-aarch64\build\bin\clang.exe" "-cc1" "-internal-isystem" "c:\buildbot\as-builder-2\llvm-clang-win-x-aarch64\build\lib\clang\11.0.0\include" "-nostdsysteminc" "-std=c++11" "-Wuninitialized" "-verify" "C:\buildbot\as-builder-2\llvm-clang-win-x-aarch64\llvm-project\clang\test\Analysis\uninit-asm-goto.cpp"<br>
# command stderr:<br>
error: 'warning' diagnostics seen but not expected: <br>
<br>
File C:\buildbot\as-builder-2\llvm-clang-win-x-aarch64\llvm-project\clang\test\Analysis\uninit-asm-goto.cpp Line 6: value size does not match register size specified by the constraint and modifier<br>
<br>
File C:\buildbot\as-builder-2\llvm-clang-win-x-aarch64\llvm-project\clang\test\Analysis\uninit-asm-goto.cpp Line 6: value size does not match register size specified by the constraint and modifier<br>
<br>
error: 'note' diagnostics seen but not expected: <br>
<br>
File C:\buildbot\as-builder-2\llvm-clang-win-x-aarch64\llvm-project\clang\test\Analysis\uninit-asm-goto.cpp Line 6: use constraint modifier "w"<br>
<br>
File C:\buildbot\as-builder-2\llvm-clang-win-x-aarch64\llvm-project\clang\test\Analysis\uninit-asm-goto.cpp Line 6: use constraint modifier "w"<br>
<br>
4 errors generated.<br>
<br>
would you fix the problem or revert the commit?<br>
<br>
<br>
Repository:<br>
rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
<a href="https://reviews.llvm.org/D69876/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D69876/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D69876" rel="noreferrer" target="_blank">https://reviews.llvm.org/D69876</a><br>
<br>
<br>
<br>
</blockquote></div>