Re: r311330 - Fixed a crash on replaying Preamble's PP conditional stack.

2017-08-23 Thread Hans Wennborg via cfe-commits
Thanks! r311591. On Wed, Aug 23, 2017 at 12:42 PM, Richard Smith wrote: > LG for branch. > > On 23 August 2017 at 11:04, Hans Wennborg via cfe-commits > wrote: >> >> Richard: ping? >> >> On Mon, Aug 21, 2017 at 1:00 PM, Hans Wennborg wrote: >> > Nikolai suggested this should be merged to 5.0. R

Re: r311330 - Fixed a crash on replaying Preamble's PP conditional stack.

2017-08-23 Thread Richard Smith via cfe-commits
LG for branch. On 23 August 2017 at 11:04, Hans Wennborg via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Richard: ping? > > On Mon, Aug 21, 2017 at 1:00 PM, Hans Wennborg wrote: > > Nikolai suggested this should be merged to 5.0. Richard, what do you > think? > > > > On Mon, Aug 21, 2017

Re: r311330 - Fixed a crash on replaying Preamble's PP conditional stack.

2017-08-23 Thread Hans Wennborg via cfe-commits
Richard: ping? On Mon, Aug 21, 2017 at 1:00 PM, Hans Wennborg wrote: > Nikolai suggested this should be merged to 5.0. Richard, what do you think? > > On Mon, Aug 21, 2017 at 5:03 AM, Ilya Biryukov via cfe-commits > wrote: >> Author: ibiryukov >> Date: Mon Aug 21 05:03:08 2017 >> New Revision: 3

Re: r311330 - Fixed a crash on replaying Preamble's PP conditional stack.

2017-08-21 Thread Hans Wennborg via cfe-commits
Nikolai suggested this should be merged to 5.0. Richard, what do you think? On Mon, Aug 21, 2017 at 5:03 AM, Ilya Biryukov via cfe-commits wrote: > Author: ibiryukov > Date: Mon Aug 21 05:03:08 2017 > New Revision: 311330 > > URL: http://llvm.org/viewvc/llvm-project?rev=311330&view=rev > Log: > F