compnerd marked 2 inline comments as done.
compnerd added inline comments.

================
Comment at: clang/test/SemaObjC/attr-swift-error.m:55
+
+// This is unconventional and very odd but we do process it.
+- (int)q0:(const NSError **)error 
__attribute__((__swift_error__(nonzero_result)));
----------------
gribozavr2 wrote:
> 
Removed the test case in light of John's comment.


================
Comment at: clang/test/SemaObjC/attr-swift-error.m:97
+
+// This is unconventional and very odd but we do process it.
+extern void *q0(const CFErrorRef *) 
__attribute__((__swift_error__(null_result)));
----------------
gribozavr2 wrote:
> 
Similar


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87331/new/

https://reviews.llvm.org/D87331

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to