<div dir="ltr">Looks fine Will, please commit but keep a comment and a link to the connect bug.  We don't want somebody to accidentally "clean-up" the code and break things again.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 27, 2015 at 7:49 AM, Will Wilson via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@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 dir="ltr">I've ended up using the simplified fix described on the MS connect bug report for this issue:<div><br></div><div><blockquote style="margin:0px 0px 0px 40px;border:none;padding:0px"><div><a href="https://connect.microsoft.com/VisualStudio/feedback/details/1741530" target="_blank">https://connect.microsoft.com/VisualStudio/feedback/details/1741530</a><br></div><div><br></div></blockquote></div><blockquote style="margin:0px 0px 0px 40px;border:none;padding:0px">I’ve been able to work around this by changing ASTContext.cpp:368 from<span class=""><br> <br>if (RC) {<br>    Raw.setRaw(RC);<br>    Raw.setKind(RawCommentAndCacheFlags::FromDecl);<br>} else<br><br></span>to<br><br>if (RC) {<br>    Raw.setKind(RawCommentAndCacheFlags::FromDecl);<br>    Raw.setRaw(RC);<br>} else</blockquote><div><br></div><div>Since this simply swaps two orthogonal calls I've checked it passes the tests and attached the patch.</div><div><br></div><div>Let me know if it's okay to commit.</div><div><br></div><div>Thanks,</div><div>Will.</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On 27 October 2015 at 11:42, Will Wilson <span dir="ltr"><<a href="mailto:will@indefiant.com" target="_blank">will@indefiant.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Building latest HEAD using:<div><br></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div>cmake -G Ninja -DCMAKE_BUILD_TYPE=RelWithDebInfo -DLLVM_USE_CRT_RELWITHDEBINFO=MT -DLLVM_BUILD_TOOLS=OFF -DLLVM_INCLUDE_EXAMPLES=OFF -DLLVM_ENABLE_ASSERTIONS=OFF LLVM_BUILD_32_BITS=ON ..\llvm</div></blockquote><div><br></div><div>VC++ version: Microsoft (R) C/C++ Optimizing Compiler Version 19.00.23026 for x86</div><div><div><br></div></div><div>Also produces the ICE mentioned before by <span style="font-size:12.8px">grimar:</span></div><div><br></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div><div>[1003/1049] Building CXX object tools\...eFiles\clangAST.dir\ASTContext.cpp.obj</div></div><div><div>FAILED: C:\PROGRA~2\MI0E91~1.0\VC\bin\cl.exe   /nologo /TP /DWIN32 /D_WINDOWS -wd4141 -wd4146 -wd4180 -wd4244 -wd4258 -wd4267 -wd4291 -wd4345 -wd4351 -wd4355 -wd4456 -wd4457 -wd4458 -wd4459 -wd4503 -wd4624 -wd4722 -wd4800 -wd4100 -wd4127 -wd4512 -wd4505 -wd4610 -wd4510 -wd4702 -wd4245 -wd4706 -wd4310 -wd4701 -wd4703 -wd4389 -wd4611 -wd4805 -wd4204 -wd4577 -wd4091 -wd4324 -w14062 -we4238 /W4 /Zc:inline /Zc:sizedDealloc- /MT /Zi /O2 /Ob1 /D NDEBUG -Itools\clang\lib\AST -IW:\rec\llvm\tools\clang\lib\AST -IW:\rec\llvm\tools\clang\include -Itools\clang\include -Iinclude -IW:\rec\llvm\include /EHs-c- /GR- /showIncludes -DCLANG_ENABLE_ARCMT -DCLANG_ENABLE_OBJC_REWRITER -DCLANG_ENABLE_STATIC_ANALYZER -DGTEST_HAS_RTTI=0 -D_CRT_NONSTDC_NO_DEPRECATE -D_CRT_NONSTDC_NO_WARNINGS -D_CRT_SECURE_NO_DEPRECATE -D_CRT_SECURE_NO_WARNINGS -D_GNU_SOURCE -D_HAS_EXCEPTIONS=0 -D_SCL_SECURE_NO_DEPRECATE -D_SCL_SECURE_NO_WARNINGS -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS /Fotools\clang\lib\AST\CMakeFiles\clangAST.dir\ASTContext.cpp.obj /Fdtools\clang\lib\AST\CMakeFiles\clangAST.dir\ /FS -c W:\rec\llvm\tools\clang\lib\AST\ASTContext.cpp</div></div><div><div>w:\rec\llvm\tools\clang\lib\ast\astcontext.cpp(334) : fatal error C1001: An internal error has occurred in the compiler.</div></div><span><div><div>(compiler file 'f:\dd\vctools\compiler\utc\src\p2\main.c', line 246)</div></div></span><span><div><div> To work around this problem, try simplifying or changing the program near the locations listed above.</div></div></span><div><div>Please choose the Technical Support command on the Visual C++</div></div><div><div> Help menu, or open the Technical Support help file for more information</div></div><div><div><br></div></div><div><div>INTERNAL COMPILER ERROR in 'C:\PROGRA~2\MI0E91~1.0\VC\bin\cl.exe'</div></div><div><div>    Please choose the Technical Support command on the Visual C++</div></div><div><div>    Help menu, or open the Technical Support help file for more information</div></div></blockquote><div><br></div><div>Did anyone find a non-invasive workaround or is the patch still the way to go?</div><div><br></div><div>Thanks,</div><div>Will.</div></div><div class="gmail_extra"><div><div><br><div class="gmail_quote">On 5 October 2015 at 13:39, Manuel Klimek via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@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">klimek added a comment.<br>
<br>
Note: with VS Professional 14.0.23107.0 D14REL I do not get this error.<br>
<div><div><br>
<br>
<a href="http://reviews.llvm.org/D13203" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13203</a><br>
<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div></div><span><font color="#888888">-- <br><div><div dir="ltr"><div><span style="background-color:rgb(255,255,255);color:rgb(68,68,68);font-family:Arial,Helvetica,sans-serif"><b>Indefiant </b>: <a href="http://www.indefiant.com" target="_blank">http://www.indefiant.com</a></span></div><div><span style="font-family:Arial,Helvetica,sans-serif"><font color="#999999">Home of Recode : Runtime C++ Editing for VS</font></span></div></div></div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr"><div><span style="background-color:rgb(255,255,255);color:rgb(68,68,68);font-family:Arial,Helvetica,sans-serif"><b>Indefiant </b>: <a href="http://www.indefiant.com" target="_blank">http://www.indefiant.com</a></span></div><div><span style="font-family:Arial,Helvetica,sans-serif"><font color="#999999">Home of Recode : Runtime C++ Editing for VS</font></span></div></div></div>
</div>
</div></div><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>