r211420 - Driver: enhance MSC version compatibility
Aaron Ballman
aaron at aaronballman.com
Tue Jun 24 07:31:30 PDT 2014
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.
~Aaron
More information about the cfe-commits
mailing list