rjmccall added inline comments.
================
Comment at: clang/lib/Sema/SemaDecl.cpp:6784
+ if (II->getInterestingIdentifierID() != 0)
+ NewTD->addAttr(AvailableOnlyInDefaultEvalMethodAttr::Create(Context));
}
----------------
Please switch over the interesting identifiers here; we don't want to assume
this feature is only used for these two names.
In fact, should we go ahead and immediately apply it to the four identifiers
above this? That would be nice, because then we could actually do this in two
patches: one patch that does the refactor to track interesting identifiers but
doesn't cause any functionality changes and a second, very small patch that
just introduces the new special treatment for `float_t` and `double_t`.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146148/new/
https://reviews.llvm.org/D146148
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
