tsan: minor IgnoreSet refactoring

[dvyukov]

Summary

1. Move kMaxSize declaration to private section.
2. Inline Reset, it's trivial and called semi-frequently.

[melver]

compiler-rt/lib/tsan/rtl/tsan_ignoreset.cpp

31–32

Could make this inlinable too?
Would save generation of IgnoreSet::Size() function even if called infrequently.

[dvyukov]

inline Size

compiler-rt/lib/tsan/rtl/tsan_ignoreset.h

 #define TSAN_IGNORESET_H
 
 #include "tsan_defs.h"
 
 namespace __tsan {
 
 class IgnoreSet {
  public:
-  static const uptr kMaxSize = 16;
-
   IgnoreSet();
   void Add(StackID stack_id);
-  void Reset();
-  uptr Size() const;
+  void Reset() { size_ = 0; }
+  uptr Size() const { return size_; }
   StackID At(uptr i) const;
 
  private:
+  static constexpr uptr kMaxSize = 16;
   uptr size_;
   StackID stacks_[kMaxSize];
 };
 
 }  // namespace __tsan
 
 #endif  // TSAN_IGNORESET_H

compiler-rt/lib/tsan/rtl/tsan_ignoreset.cpp

void IgnoreSet::Add(StackID stack_id) {
   if (size_ == kMaxSize)
     return;
   for (uptr i = 0; i < size_; i++) {
     if (stacks_[i] == stack_id)
       return;
   }
   stacks_[size_++] = stack_id;
 }
 
-void IgnoreSet::Reset() {
-  size_ = 0;
-}
-
-uptr IgnoreSet::Size() const {
-  return size_;
-}
-
 StackID IgnoreSet::At(uptr i) const {
   CHECK_LT(i, size_);
   CHECK_LE(size_, kMaxSize);
   return stacks_[i];
 }
 
 }  // namespace __tsan