summaryrefslogtreecommitdiffstats
path: root/clang/test
diff options
context:
space:
mode:
authorArtem Dergachev <artem.dergachev@gmail.com>2019-04-26 02:05:12 +0000
committerArtem Dergachev <artem.dergachev@gmail.com>2019-04-26 02:05:12 +0000
commitb591845f4b482a2aec74f121f5f03da2ebefcef4 (patch)
treeb495ff810f852e39bde353bc250224a6ff192d42 /clang/test
parente1320b0ed06c2c5df30713816a8da6acd814568b (diff)
downloadbcm5719-llvm-b591845f4b482a2aec74f121f5f03da2ebefcef4.tar.gz
bcm5719-llvm-b591845f4b482a2aec74f121f5f03da2ebefcef4.zip
[analyzer] Fix crash when returning C++ objects from ObjC messages-to-nil.
the assertion is in fact incorrect: there is a cornercase in Objective-C++ in which a C++ object is not constructed with a constructor, but merely zero-initialized. Namely, this happens when an Objective-C message is sent to a nil and it is supposed to return a C++ object. Differential Revision: https://reviews.llvm.org/D60988 llvm-svn: 359262
Diffstat (limited to 'clang/test')
-rw-r--r--clang/test/Analysis/nil-receiver.mm24
1 files changed, 24 insertions, 0 deletions
diff --git a/clang/test/Analysis/nil-receiver.mm b/clang/test/Analysis/nil-receiver.mm
new file mode 100644
index 00000000000..c462fce1828
--- /dev/null
+++ b/clang/test/Analysis/nil-receiver.mm
@@ -0,0 +1,24 @@
+// RUN: %clang_analyze_cc1 -analyzer-checker=core,debug.ExprInspection \
+// RUN: -verify %s
+
+#define nil ((id)0)
+
+void clang_analyzer_eval(int);
+
+struct S {
+ int x;
+ S();
+};
+
+@interface I
+@property S s;
+@end
+
+void foo() {
+ // This produces a zero-initialized structure.
+ // FIXME: This very fact does deserve the warning, because zero-initialized
+ // structures aren't always valid in C++. It's particularly bad when the
+ // object has a vtable.
+ S s = ((I *)nil).s;
+ clang_analyzer_eval(s.x == 0); // expected-warning{{TRUE}}
+}
OpenPOWER on IntegriCloud