[PATCH] D17893: Sema: Add semantic analysis for the C++ ABI stability attributes and whitelist.
Mehdi AMINI via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 16 15:04:03 PDT 2016
joker.eph added inline comments.
================
Comment at: lib/Sema/SemaDeclCXX.cpp:4999
@@ +4998,3 @@
+ return;
+ }
+
----------------
pcc wrote:
> joker.eph wrote:
> > joker.eph wrote:
> > > Isn't this correct by the loop which starts with `DC = InnermostExternalDC`?
> > `s/correct/already covered/`
> That loop checks for glob contexts (contexts ending in `**` in the ABI list), which have different semantics to regular contexts (ending in `*`).
Oh, I misread `UnstableABIGlobContexts` for `UnstableABIContexts`...
http://reviews.llvm.org/D17893
More information about the cfe-commits
mailing list