[LLDB][NFC] Fix LLDB_WATCH_TYPE_IS_VALID macro

[fixathon]

Summary

LLDB_WATCH_TYPE_IS_VALID would always return true when validating watchpoint type
by using bitwise-or instead of bitwise-and to apply validation flags.

[DavidSpickett]

Sometimes you wonder how anything works :)

I looked at the places this is used and I don't see a way to reach them from the API without the type already being LLDB_WATCH_TYPE_READ or LLDB_WATCH_TYPE_WRITE. It's checked in lldb/source/Target/Target.cpp:CreateWatchpoint but there's no way to fail that check using existing calls, only direct from C++. So adding a test for this isn't possible.

LGTM.

lldb/include/lldb/lldb-defines.h

 #define LLDB_BREAK_ID_IS_INTERNAL(bid) ((bid) < 0)
 
 // Watchpoints
 #define LLDB_INVALID_WATCH_ID 0
 #define LLDB_WATCH_ID_IS_VALID(uid) ((uid) != (LLDB_INVALID_WATCH_ID))
 #define LLDB_WATCH_TYPE_READ (1u << 0)
 #define LLDB_WATCH_TYPE_WRITE (1u << 1)
 #define LLDB_WATCH_TYPE_IS_VALID(type)                                         \
-  ((type | LLDB_WATCH_TYPE_READ) || (type | LLDB_WATCH_TYPE_WRITE))
+  ((type & LLDB_WATCH_TYPE_READ) || (type & LLDB_WATCH_TYPE_WRITE))
 
 // Generic Register Numbers
 #define LLDB_REGNUM_GENERIC_PC 0    // Program Counter
 #define LLDB_REGNUM_GENERIC_SP 1    // Stack Pointer
 #define LLDB_REGNUM_GENERIC_FP 2    // Frame Pointer
 #define LLDB_REGNUM_GENERIC_RA 3    // Return Address
 #define LLDB_REGNUM_GENERIC_FLAGS 4 // Processor flags register
 #define LLDB_REGNUM_GENERIC_ARG1                                               \