[memprof] Fix RAII usage of SpinMutexLock, remove unused headers.

[snehasish]

Summary

For the RAII lock usage we need to create a local var. There were some headers which clang-tidy identified as unused.

[tejohnson]

lgtm

compiler-rt/lib/memprof/memprof_allocator.cpp

 #include "sanitizer_common/sanitizer_allocator_checks.h"
 #include "sanitizer_common/sanitizer_allocator_interface.h"
 #include "sanitizer_common/sanitizer_allocator_report.h"
 #include "sanitizer_common/sanitizer_errno.h"
 #include "sanitizer_common/sanitizer_file.h"
 #include "sanitizer_common/sanitizer_flags.h"
 #include "sanitizer_common/sanitizer_interface_internal.h"
 #include "sanitizer_common/sanitizer_internal_defs.h"
-#include "sanitizer_common/sanitizer_list.h"
 #include "sanitizer_common/sanitizer_procmaps.h"
 #include "sanitizer_common/sanitizer_stackdepot.h"
-#include "sanitizer_common/sanitizer_vector.h"
 
 #include <sched.h>
 #include <time.h>
 
 // Allow the user to specify a profile output file via the binary.
 SANITIZER_WEAK_ATTRIBUTE char __memprof_profile_filename[1];
 
 namespace __memprof {
struct Allocator {
 
   ~Allocator() {
     atomic_store_relaxed(&destructing, 1);
     FinishAndWrite();
   }
 
   static void PrintCallback(const uptr Key, LockedMemInfoBlock *const &Value,
                             void *Arg) {
-    SpinMutexLock(&Value->mutex);
+    SpinMutexLock l(&Value->mutex);
     Print(Value->mib, Key, bool(Arg));
   }
 
   void FinishAndWrite() {
     // Use profile name specified via the binary itself if it exists, and hasn't
     // been overrriden by a flag at runtime.
     if (__memprof_profile_filename[0] != 0 && !common_flags()->log_path)
       __sanitizer_set_report_path(__memprof_profile_filename);