aprantl added a comment.

Code LGTM, it would be nice if we could further reduce the test case.



================
Comment at: clang/test/CodeGenObjC/objc-arc-ubsan-debugging.m:2
+// RUN: %clang -x objective-c -target arm64-apple-macos12.0 -fobjc-arc 
-std=gnu99  -O0 -fsanitize=undefined -fsanitize=nullability -c %s -v -g
+#include <Foundation/Foundation.h>
+
----------------
Could you get by with just defining NSObject inline and forward-declaring 
NSString? Including Foundation.h is very expensive.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142243

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

Reply via email to