Re: r266775 - Pass dwarf-version to cc1as.

2016-08-16 Thread Dimitry Andric via cfe-commits
Ah, I missed Doug's follow-up commit r266797, which moved the code again. I noticed that the RenderDebugEnablingArgs() call was missing in the spot indicated below, but it was only moved. -Dimitry > On 17 Aug 2016, at 01:17, Hans Wennborg wrote: > > Looks like it was committed back in April,

Re: r266775 - Pass dwarf-version to cc1as.

2016-08-16 Thread Hans Wennborg via cfe-commits
Looks like it was committed back in April, so unless I'm missing something it should already be in the branch (which was cut mid-July). Thanks, Hans On Tue, Aug 16, 2016 at 2:16 PM, Dimitry Andric wrote: > Hi Hans, > > Can we please merge this one to release_39 too? I forgot all about it, sorry

Re: r266775 - Pass dwarf-version to cc1as.

2016-08-16 Thread Dimitry Andric via cfe-commits
Hi Hans, Can we please merge this one to release_39 too? I forgot all about it, sorry. Doug, can you please approve? -Dimitry > On 19 Apr 2016, at 19:43, Douglas Katzman via cfe-commits > wrote: > > Author: dougk > Date: Tue Apr 19 12:43:54 2016 > New Revision: 266775 > > URL: http://llvm.