[PATCH] D125780: [llvm-debuginfo-analyzer] 05 - Select elements

[Paul Robinson via Phabricator via llvm-commits]

probinson added inline comments.


================
Comment at: llvm/include/llvm/DebugInfo/LogicalView/Core/LVOptions.h:556
+
+public:
+  LVPatterns() {
----------------
No need to repeat `public:`


================
Comment at: llvm/include/llvm/DebugInfo/LogicalView/Core/LVOptions.h:634
+
+public:
+  // Conditions to print an object.
----------------
No need to repeat `public:`


================
Comment at: llvm/include/llvm/DebugInfo/LogicalView/Core/LVSymbol.h:163
+
 public:
   void print(raw_ostream &OS, bool Full = true) const override;
----------------
No need to repeat `public:`


================
Comment at: llvm/unittests/DebugInfo/LogicalView/SelectElementsTest.cpp:54
+    T *Element = new (std::nothrow) T();
+    EXPECT_NE(Element, nullptr);
+    (Element->*Function)();
----------------
Use `ASSERT_NE` to bail out of the test if the allocation fails.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125780/new/

https://reviews.llvm.org/D125780