Re: r292632 - Fix actually-reachable llvm_unreachable.

2017-01-24 Thread Richard Smith via cfe-commits
On 23 January 2017 at 08:57, David Blaikie wrote: > Should this test that some specific mangling comes out the other end > (tests that amount to "test that this does anything other than crashing" > always make me a bit suspicious that there's /some/ specific behavior that > is intended/should be

Re: r292632 - Fix actually-reachable llvm_unreachable.

2017-01-23 Thread David Blaikie via cfe-commits
Should this test that some specific mangling comes out the other end (tests that amount to "test that this does anything other than crashing" always make me a bit suspicious that there's /some/ specific behavior that is intended/should be tested for) On Fri, Jan 20, 2017 at 11:01 AM Richard Smith

r292632 - Fix actually-reachable llvm_unreachable.

2017-01-20 Thread Richard Smith via cfe-commits
Author: rsmith Date: Fri Jan 20 12:50:12 2017 New Revision: 292632 URL: http://llvm.org/viewvc/llvm-project?rev=292632&view=rev Log: Fix actually-reachable llvm_unreachable. Modified: cfe/trunk/lib/AST/ItaniumMangle.cpp cfe/trunk/test/CodeGenCXX/mangle.cpp Modified: cfe/trunk/lib/AST/Ita