Re: r347035 - [clang] - Simplify tools::SplitDebugName.

2018-11-27 Thread David Blaikie via cfe-commits
Just copying some of the information given through Phab ( https://reviews.llvm.org/rL347035 ) - this patch was reverted in r347676. Would be great to get that fixed/recommitted! On Fri, Nov 16, 2018 at 12:01 AM George Rimar via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: grimar > D

r347035 - [clang] - Simplify tools::SplitDebugName.

2018-11-16 Thread George Rimar via cfe-commits
Author: grimar Date: Thu Nov 15 23:59:24 2018 New Revision: 347035 URL: http://llvm.org/viewvc/llvm-project?rev=347035&view=rev Log: [clang] - Simplify tools::SplitDebugName. This should be NFC change. SplitDebugName recently started to accept the `Output` that can be used to simplify the logic