[analyzer] Move test case to existing test file and remove duplicated test file.

[ASDenysPetrov]

Summary

Move the test case to existing test file. Remove test file as duplicated. The file was mistakenly added due to concerns of a hidden bug (see D104381). After it turned out, that the bug was already fixed with another revision (D85817) and corresponding test was added as well, we can remove this file.

[NoQ]

I have no opinion here. There's nothing wrong with having more tests and they don't look like exact duplicates of each other(?)

[ASDenysPetrov]

In D106152#2885091, @NoQ wrote:

I have no opinion here. There's nothing wrong with having more tests and they don't look like exact duplicates of each other(?)

OK, let's just make a compromise and move this test into existing one instead of removing. WDYT?

[ASDenysPetrov]

Moved the test case to existing file.

[ASDenysPetrov]

@NoQ just a ping

[ASDenysPetrov]

I think the patch is pretty safe, then I decided to load it.

clang/test/Analysis/PR46264.cpp

int testIndirectCrash() {
   int *x = 0;
   int A::*cp = &A::c;
 
   if (cp)      // no crash
     return *x; // expected-warning{{Dereference of null pointer (loaded from variable 'x')}}
 
   return 10;
 }
+
+// PR46264
+// This case shall not crash with an assertion failure about void* dereferening.
+namespace ns1 {
+namespace a {
+class b {
+public:
+  typedef int b::*c;
+  operator c() { return d ? &b::d : 0; }
+  int d;
+};
+} // namespace a
+using a::b;
+class e {
+  void f();
+  void g();
+  b h;
+};
+void e::f() {
+  e *i;
+  if (h)
+    i->g(); // expected-warning{{Called C++ object pointer is uninitialized}}
+}
+} // namespace ns1

clang/test/Analysis/diagnostics/PR46264.cpp

-// RUN: %clang_analyze_cc1 -analyzer-checker=core -analyzer-output=text -verify %s
-
-// PR46264
-// This case shall not crash with an assertion failure about void* dereferening.
-// The crash has been last seen on commit
-// `3ed8ebc2f6b8172bed48cc5986d3b7af4cfca1bc` from 24.05.2020.
-namespace ns1 {
-namespace a {
-class b {
-public:
-  typedef int b::*c;
-  operator c() { return d ? &b::d : 0; }
-  // expected-note@-1{{'?' condition is true}}
-  // expected-note@-2{{Assuming field 'd' is not equal to 0}}
-  // expected-note@-3{{Returning value, which participates in a condition later}}
-  int d;
-};
-} // namespace a
-using a::b;
-class e {
-  void f();
-  void g();
-  b h;
-};
-void e::f() {
-  e *i;
-  // expected-note@-1{{'i' declared without an initial value}}
-  if (h)
-    // expected-note@-1{{Taking true branch}}
-    // expected-note@-2{{'b::operator int ns1::a::b::*'}}
-    // expected-note@-3{{Returning from 'b::operator int ns1::a::b::*'}}
-    i->g();
-  // expected-note@-1{{Called C++ object pointer is uninitialized}}
-  // expected-warning@-2{{Called C++ object pointer is uninitialized}}
-}
-} // namespace ns1