friss added a comment.

This looks good outside of 2 cross-platform issues in the tests themselves.



================
Comment at: lldb/test/API/macosx/macCatalyst/TestMacCatalyst.py:27
+        self.expect("image list -t -b",
+                    patterns=["x86_64.*-apple-ios.*-macabi a\.out"])
+        self.expect("fr v s", "Hello macCatalyst")
----------------
hardcoded architecture and no architecture decorator.


================
Comment at: 
lldb/test/API/macosx/macCatalystAppMacOSFramework/TestMacCatalystAppWithMacOSFramework.py:27-28
+        self.expect("image list -t -b",
+                    patterns=["x86_64.*-apple-ios.*-macabi a\.out",
+                              "x86_64.*-apple-macosx.* libfoo.dylib[^(]"])
+        self.expect("fr v s", "Hello macCatalyst")
----------------
ditto


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

https://reviews.llvm.org/D84576

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

Reply via email to