Re: r313729 - Implement C++ [basic.link]p8.

2018-03-02 Thread Alex L via cfe-commits
Hi Richard, This commit has caused a second regression in Clang which triggers a new invalid -Wunused-function warning. I filed https://bugs.llvm.org/show_bug.cgi?id=36584 which contains the minimized test case for this issue. Do you mind taking a look at it? Thanks, Alex On 19 December 2017 at

Re: r313729 - Implement C++ [basic.link]p8.

2017-12-19 Thread Alex L via cfe-commits
Hi Richard, This commit has caused a new regression in Clang which causes an assertion failure for extern "C" function definitions whose return type is a pointer to a record. I filed https://bugs.llvm.org/show_bug.cgi?id=35697 which contains the minimized test case for this issue. Do you mind taki

r313729 - Implement C++ [basic.link]p8.

2017-09-20 Thread Richard Smith via cfe-commits
Author: rsmith Date: Wed Sep 20 00:22:00 2017 New Revision: 313729 URL: http://llvm.org/viewvc/llvm-project?rev=313729&view=rev Log: Implement C++ [basic.link]p8. If a function or variable has a type with no linkage (and is not extern "C"), any use of it requires a definition within the same tran