labath added a comment.

No opinion on the code, but I think the test needs to be cleaned up _a lot_.



================
Comment at: lldb/test/Shell/Settings/Inputs/main.cpp:1
+//===-- main.cpp ------------------------------------------------*- C++ 
-*-===//
+//
----------------
This file is about 70 lines longer than necessary to test this functionality, 
and it contains a #include statement in the middle of the file which is 
completely bogus


================
Comment at: lldb/test/Shell/Settings/TestFrameFormatMangling.test:2
+# UNSUPPORTED: system-windows
+# RUN: %clangxx_host -std=c++14 -g -O0 %S/Inputs/main.cpp -o %t.out
+# RUN: %lldb -x -b -s %s %t.out | FileCheck %s
----------------
The -std argument doesn't seem relevant here.


================
Comment at: lldb/test/Shell/Settings/TestFrameFormatMangling.test:3
+# RUN: %clangxx_host -std=c++14 -g -O0 %S/Inputs/main.cpp -o %t.out
+# RUN: %lldb -x -b -s %s %t.out | FileCheck %s
+br set -p "Set break"
----------------
-x not needed


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71237



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to