r218925 - Patch to warn if 'override' is missing

jahanian fjahanian at apple.com
Fri Oct 3 09:43:11 PDT 2014


On Oct 3, 2014, at 1:12 AM, Alexander Potapenko <glider at google.com> wrote:

> I'm going to revert this patch because it broke bootstraping on both
> Linux and Mac.

Ok. But this is strange. Before checking in, I made sure there is not a single warning building llvm with this patch
(I had already added the ‘override’ to llvm source before this patch where was needed). But this is going to happen with every new warning and
maybe we should add the warning where needed instead of revert.
- Fariborz






More information about the cfe-commits mailing list