[PATCH] D46747: [Sema] Use dotted form of macOS version for unguarded availability FixIts

[Erik Pilkington via Phabricator via cfe-commits]

erik.pilkington added a comment.

> I am now thinking about just switching to dot format in ParseAvailabilityAttribute() because it's much simpler to maintain consistency that way.

Okay, but if we're going to treat underscores as dots while printing diagnostics and fixits (which I think we should), then we shouldn't track the spelling in VersionTuple: we no longer have any use for it. The only reason that VersionTuple supports this is because of r219124, which was written so that diagnostic printing would be consistent with how the attribute was spelled. Maybe there was a good reason for this, but I can't see the radar that was linked in the commit message. Can you look up that radar to see what r219124 was actually trying to fix? I don't think we should move forward with this until we have that context.

Thanks! 
Erik


https://reviews.llvm.org/D46747