Revised test to pass under updated dtor callback implementation

[nmusgrave]

Summary

New implementation for dtor sanitizer callback poisons only class members, and emits poisoning callback before base dtor invoked.

[eugenis]

test/msan/dtor-base-access.cc

48

Why are you removing the destructor call?
That's exactly what this test is about.

[nmusgrave]

• Explicit dtor invocation

[eugenis]

LGTM

test/msan/dtor-base-access.cc

 // RUN: %clangxx_msan %s -O0 -fsanitize=memory -fsanitize-memory-use-after-dtor -o %t && MSAN_OPTIONS=poison_in_dtor=1 %run %t >%t.out 2>&1
 
 // RUN: %clangxx_msan %s -O1 -fsanitize=memory -fsanitize-memory-use-after-dtor -o %t && MSAN_OPTIONS=poison_in_dtor=1 %run %t >%t.out 2>&1
 
 // RUN: %clangxx_msan %s -O2 -fsanitize=memory -fsanitize-memory-use-after-dtor -o %t && MSAN_OPTIONS=poison_in_dtor=1 %run %t >%t.out 2>&1
 
-// XFAIL: *
-
 #include <sanitizer/msan_interface.h>
 #include <assert.h>
 
 class Base {
  public:
   int *x_ptr;
   Base(int *y_ptr) {
     // store value of subclass member
Derived::~Derived() {
   assert(__msan_test_shadow(&this->y, sizeof(this->y)) == -1);
   // ok access base class members
   assert(__msan_test_shadow(&this->x_ptr, sizeof(this->x_ptr)) == -1);
 }
 
 int main() {
   Derived *d = new Derived();
   assert(__msan_test_shadow(&d->x_ptr, sizeof(d->x_ptr)) == -1);
   d->~Derived();
   assert(__msan_test_shadow(&d->x_ptr, sizeof(d->x_ptr)) != -1);
   return 0;
 }