Yeah, let's drop it. Sorry for the runaround.
On Mon, Jun 30, 2014 at 8:26 AM, Ehsan Akhgari <[email protected]> wrote: > We hit this error in our code and I have since fixed the code on our > side. I'm not aware of any system headers or generate code that hits this. > > Based on that, should we can this patch? > > -- > Ehsan > <http://ehsanakhgari.org/> > > > On Sat, Jun 28, 2014 at 2:07 PM, Reid Kleckner <[email protected]> wrote: > >> I'd echo what David said, but otherwise, this seems fine. >> >> ================ >> Comment at: lib/Sema/SemaDecl.cpp:3290 >> @@ +3289,3 @@ >> + Diag(SS.getBeginLoc(), >> + getLangOpts().MicrosoftExt ? >> + diag::ext_standalone_class_nested_name_specifier : >> ---------------- >> We use LangOpts.MSVCCompat to control these language bending extensions. >> >> http://reviews.llvm.org/D4333 >> >> >> >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
