(original) (raw)
On Wed, Jul 15, 2015 at 10:45 AM, Russell Wallace <russell.wallace@gmail.com> wrote:
Basic test results on Windows 7, visual studio 2013 (64 bit):Build clang with visual studio - okayBuild clang with itself - okayBuild Python - okayBuild Ruby - fails on conftest.c, but 3.6 also failed so this is not a regression bugBuild Perl - fails. 3.6 also failed, but I think the error message was different, so this could be a regression bug but hopefully it's actually an improvement. Current error message:cl -c -I. -nologo -GF -W3 -I..\\lib\\CORE -I.\\include -I. -I.. -DWIN32 -D\_CONSOLE -DNO\_STRICT -DWIN64 -DCONSERVATIVE -D\_CRT\_SECURE\_NO\_DEPRECATE -D\_CRT\_NONSTDC\_NO\_DEPRECATE -DPERLDLL -DPERL\_CORE-O1 -MD -Zi -DNDEBUG -GL -fp:precise -DPERL\_TEXTMODE\_SCRIPTS -DPERL\_IMPLICIT\_CONTEXT -DPERL\_IMPLICIT\_SYS -TP -EHsc -Foperllib.obj perllib.cclang-cl.exe: warning: argument unused during compilation: '-GL'In file included from perllib.c:10:In file included from ..\\lib\\CORE\\perl.h:3060:In file included from .\\win32thread.h:4:./win32.h(284,25) : error: 'selectany' can only be applied to data items with external linkage
That line is:
extern const \_\_declspec(selectany) union { unsigned \_\_int64 \_\_q; double \_\_d; } \_\_PL\_nan\_u = { 0x7FF8000000000000UI64 };
If it's written like so, clang-cl accepts it:
union U { unsigned \_\_int64 \_\_q; double \_\_d; };
extern const \_\_declspec(selectany) U \_\_PL\_nan\_u = { 0x7FF8000000000000UI64 };
I guess cl.exe applies the declspec to \_\_PL\_nan\_u while we try to apply it to the type? (Even though it's written before "union", so according to https://msdn.microsoft.com/en-us/library/dabb5z75.aspx it should apply to the variable.) Is there a bug filed for this?
On Wed, Jul 15, 2015 at 1:25 AM, Hans Wennborg <hans@chromium.org> wrote:Hi all,
The 3.7 release branch was created from trunk at r242221 today (around
10:40 pm UTC).
Branch policy:
\- Any doc changes can go in. Updates to the release notes are highly
encouraged, and should be committed directly to the branch.
\- All other patches should be approved by the release manager (me) and
the appropriate code owner. To get a change merged, commit it to
trunk, and then reply to the commit email with myself and the code
owner cc'd, asking for approval.
\- Fixes to complete existing features may be merged. However, the
features must be completed before Phase II of testing starts,
otherwise they should be disabled. If you recently committed something
experimental to trunk, please make sure it's disabled on the branch.
\- For any bug fixes that you think might apply to the release branch,
please cc me on the commit message.
Cheers,
Hans
\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_
LLVM Developers mailing list
LLVMdev@cs.uiuc.edu http://llvm.cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_
lldb-dev mailing list
lldb-dev@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev