This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. fixathon marked an inline comment as done. Closed by commit rG06ff46d2d77f: [LLDB][NFC] Fix suspicious bitwise expression in PrintBTEntry() (authored by fixathon).
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131312/new/ https://reviews.llvm.org/D131312 Files: lldb/tools/intel-features/intel-mpx/cli-wrapper-mpxtable.cpp Index: lldb/tools/intel-features/intel-mpx/cli-wrapper-mpxtable.cpp =================================================================== --- lldb/tools/intel-features/intel-mpx/cli-wrapper-mpxtable.cpp +++ lldb/tools/intel-features/intel-mpx/cli-wrapper-mpxtable.cpp @@ -63,7 +63,7 @@ const lldb::addr_t one_cmpl64 = ~((lldb::addr_t)0); const lldb::addr_t one_cmpl32 = ~((uint32_t)0); - if ((lbound == one_cmpl64 || one_cmpl32) && ubound == 0) { + if ((lbound == one_cmpl64 || lbound == one_cmpl32) && ubound == 0) { result.Printf("Null bounds on map: pointer value = 0x%" PRIu64 "\n", value); } else { result.Printf(" lbound = 0x%" PRIu64 ",", lbound);
Index: lldb/tools/intel-features/intel-mpx/cli-wrapper-mpxtable.cpp =================================================================== --- lldb/tools/intel-features/intel-mpx/cli-wrapper-mpxtable.cpp +++ lldb/tools/intel-features/intel-mpx/cli-wrapper-mpxtable.cpp @@ -63,7 +63,7 @@ const lldb::addr_t one_cmpl64 = ~((lldb::addr_t)0); const lldb::addr_t one_cmpl32 = ~((uint32_t)0); - if ((lbound == one_cmpl64 || one_cmpl32) && ubound == 0) { + if ((lbound == one_cmpl64 || lbound == one_cmpl32) && ubound == 0) { result.Printf("Null bounds on map: pointer value = 0x%" PRIu64 "\n", value); } else { result.Printf(" lbound = 0x%" PRIu64 ",", lbound);
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits