<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">Hi Ben,<br>
I don't think your patch is compatible with MSVC. MSVC accepts
this code and clang should accept it and generate proper LLVM IR.
Actually, you can run into similar code in MSVC system headers.
That's the main problem<br>
<pre class="moz-signature" cols="72">Best regards,
Alexey Bataev
=============
Software Engineer
Intel Compiler Team
Intel Corp. </pre>
11.02.2015 20:37, Ben Langmuir пишет:<br>
</div>
<blockquote
cite="mid:%3CD464E701-0272-419E-8AB6-A23596A58CFA@apple.com%3E"
type="cite">
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
Ping
<div class=""><br class="">
<div>
<blockquote type="cite" class="">
<div class="">On Jan 26, 2015, at 9:08 AM, Ben Langmuir <<a
moz-do-not-send="true" href="mailto:blangmuir@apple.com"
class="">blangmuir@apple.com</a>> wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<meta http-equiv="Content-Type" content="text/html;
charset=UTF-8" class="">
<div style="word-wrap: break-word; -webkit-nbsp-mode:
space; -webkit-line-break: after-white-space;" class="">This
patch diagnoses a missing ‘typename’ keyword on nested
template types like A<T>::B<U>, to fix <a
moz-do-not-send="true" href="http://llvm.org/pr16909"
class="">llvm.org/pr16909</a>. In addition to fixing
an accepts-invalid, in C++11 such types would cause
assertion failures and/or invalid LLVM IR when used with
‘auto’.
<div class=""><br class="">
</div>
<div class="">I’m not 100% sure if the changes to
test/CXX/basic/basic.lookup/basic.lookup.qual/class.qual/p2.cpp
are desirable, or if we should suppress the missing
‘typename’ diagnostic when we’re already recovering on
X<T>::X<T>. I’m open to suggestions :-)</div>
<div class=""><br class="">
</div>
<div class="">Ben</div>
<div class=""><br class="">
</div>
</div>
<span
id="cid:F07AC0C3-0C93-4BAC-9DA0-7121ACA68F5D@apple.com"><pr16909.patch></span>
<meta http-equiv="Content-Type" content="text/html;
charset=UTF-8" class="">
</div>
</blockquote>
</div>
<br class="">
</div>
</blockquote>
<br>
</body>
</html>