r211420 - Driver: enhance MSC version compatibility

[Saleem Abdulrasool]

On Jun 24, 2014, at 7:31 AM, Aaron Ballman <aaron at aaronballman.com> wrote:

> On Tue, Jun 24, 2014 at 10:28 AM, Eric Christopher <echristo at gmail.com> wrote:
>> On Mon, Jun 23, 2014 at 1:02 PM, Reid Kleckner <rnk at google.com> wrote:
>>> On Sat, Jun 21, 2014 at 11:32 AM, Alp Toker <alp at nuanti.com> wrote:
>>>> 
>>>> Please split out and name the new dotted form something like
>>>> "-msc-full-version" to avoid ambiguity.
>>> 
>>> 
>>> I was inclinded to overload the option because it means we don't have to
>>> diagnose this case: -fmsc-version -fmsc-full-version=17.00.0.  We just do
>>> the obvious thing of "last one always wins".  Diagnosing the conflict isn't
>>> too hard, though, so let's just split the options and do exactly that in the
>>> driver.
>>> 
>> 
>> I'm inclined to agree with this. We could also even deprecate the
>> original option and have the new option have exactly the semantics we
>> want. Aaron?
> 
> If we deprecate the original option and warn when the options are
> combined in the meantime, then I'd be okay with splitting the options.

I think this makes it more palatable for me as well.

This just leaves the question of bike shedding: -fmsc-version-ex? :-)

> ~Aaron
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

-- 
Saleem Abdulrasool
abdulras (at) fb (dot) com