================
@@ -1191,12 +1191,18 @@ void Sema::ProcessAPINotes(Decl *D) {
if (auto CXXMethod = dyn_cast<CXXMethodDecl>(D)) {
if (!isa<CXXConstructorDecl>(CXXMethod) &&
!isa<CXXDestructorDecl>(CXXMethod) &&
- !isa<CXXConversionDecl>(CXXMethod) &&
- !CXXMethod->isOverloadedOperator()) {
+ !isa<CXXConversionDecl>(CXXMethod)) {
for (auto Reader : APINotes.findAPINotes(D->getLocation())) {
if (auto Context = UnwindTagContext(TagContext, APINotes)) {
- auto Info =
- Reader->lookupCXXMethod(Context->id, CXXMethod->getName());
+ std::string MethodName;
+ if (CXXMethod->isOverloadedOperator())
+ MethodName =
+ std::string("operator") +
+ getOperatorSpelling(CXXMethod->getOverloadedOperator());
+ else
+ MethodName = CXXMethod->getName();
----------------
compnerd wrote:
Personally, I find that a ternary is easier to read:
```
```
```suggestion
std::string MethodName = CXXMethod->isOverloadedOperator()
? std::string("operator") +
getOperatorSpelling(CXXMethod->getOverloadedOperator())
: CXXMethod->getName();
```
https://github.com/llvm/llvm-project/pull/177382
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
