[llvm-branch-commits] [llvm] [ObjC] Support objc_claimAutoreleasedReturnValue (PR #138696)

[Florian Hahn via llvm-branch-commits]

================
@@ -160,10 +160,7 @@ static bool lowerObjCCall(Function &F, const char *NewFn,
     auto *CB = cast<CallBase>(U.getUser());
 
     if (CB->getCalledFunction() != &F) {
-      objcarc::ARCInstKind Kind = objcarc::getAttachedARCFunctionKind(CB);
-      (void)Kind;
-      assert((Kind == objcarc::ARCInstKind::RetainRV ||
-              Kind == objcarc::ARCInstKind::UnsafeClaimRV) &&
+      assert((*objcarc::getAttachedARCFunction(CB) == &F) &&
----------------
fhahn wrote:

I *think* you don't need to dereference the std::optional for the compare, 
below should work

```suggestion
      assert(objcarc::getAttachedARCFunction(CB) == &F &&
```

https://github.com/llvm/llvm-project/pull/138696
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits