<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Sorry for that, it seems I overlooked those failure mails. Thanks!<div class=""><div><br class=""><blockquote type="cite" class=""><div class="">On 20 Jun 2020, at 23:22, Eric Christopher <<a href="mailto:echristo@gmail.com" class="">echristo@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">This is failing on some of the debian buildbots so I've temporarily reverted it here:<div class=""><br class=""></div><div class="">commit 10b43541360efb35a1d33e9cf1e93023ebd69b15 (HEAD -> master, origin/master, origin/HEAD)<br class="">Author: Eric Christopher <<a href="mailto:echristo@gmail.com" class="">echristo@gmail.com</a>><br class="">Date: Sat Jun 20 14:21:42 2020<br class=""><br class=""> Temporarily Revert "[lldb][NFC] Add more test for builtin formats"<br class=""> as it's failing on the debian buildbots:<br class=""><br class=""> <a href="http://lab.llvm.org:8011/builders/lldb-x86_64-debian/builds/12531" class="">http://lab.llvm.org:8011/builders/lldb-x86_64-debian/builds/12531</a><br class=""><br class=""> This reverts commit 90c1af106a20785ffd01c0d6a41db8bc0160fd11.<br class=""></div><div class=""><br class=""></div><div class="">buildbot link:</div><div class=""><br class=""></div><div class=""><a href="http://lab.llvm.org:8011/builders/lldb-x86_64-debian/builds/12531" class="">http://lab.llvm.org:8011/builders/lldb-x86_64-debian/builds/12531</a><br class=""></div><div class=""><br class=""></div><div class="">Sorry for any inconvenience.</div><div class=""><br class=""></div><div class="">-eric</div></div><br class=""><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Jun 20, 2020 at 12:35 PM Raphael Isemann via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org" class="">lldb-commits@lists.llvm.org</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br class="">
Author: Raphael Isemann<br class="">
Date: 2020-06-20T19:31:40+02:00<br class="">
New Revision: 90c1af106a20785ffd01c0d6a41db8bc0160fd11<br class="">
<br class="">
URL: <a href="https://github.com/llvm/llvm-project/commit/90c1af106a20785ffd01c0d6a41db8bc0160fd11" rel="noreferrer" target="_blank" class="">https://github.com/llvm/llvm-project/commit/90c1af106a20785ffd01c0d6a41db8bc0160fd11</a><br class="">
DIFF: <a href="https://github.com/llvm/llvm-project/commit/90c1af106a20785ffd01c0d6a41db8bc0160fd11.diff" rel="noreferrer" target="_blank" class="">https://github.com/llvm/llvm-project/commit/90c1af106a20785ffd01c0d6a41db8bc0160fd11.diff</a><br class="">
<br class="">
LOG: [lldb][NFC] Add more test for builtin formats<br class="">
<br class="">
The previous tests apparently missed a few code branches in DumpDataExtractor<br class="">
code. Also renames the 'test_instruction' which had the same name as another<br class="">
test (and Python therefore ignored the test entirely).<br class="">
<br class="">
Added: <br class="">
<br class="">
<br class="">
Modified: <br class="">
lldb/test/API/functionalities/data-formatter/builtin-formats/TestBuiltinFormats.py<br class="">
<br class="">
Removed: <br class="">
<br class="">
<br class="">
<br class="">
################################################################################<br class="">
diff --git a/lldb/test/API/functionalities/data-formatter/builtin-formats/TestBuiltinFormats.py b/lldb/test/API/functionalities/data-formatter/builtin-formats/TestBuiltinFormats.py<br class="">
index 9fefae6bbf5c..1a413a13986a 100644<br class="">
--- a/lldb/test/API/functionalities/data-formatter/builtin-formats/TestBuiltinFormats.py<br class="">
+++ b/lldb/test/API/functionalities/data-formatter/builtin-formats/TestBuiltinFormats.py<br class="">
@@ -42,6 +42,9 @@ def test(self):<br class="">
self.assertIn("= 0\n", self.getFormatted("float", "0"))<br class="">
self.assertIn("= 2\n", self.getFormatted("float", "0x40000000"))<br class="">
self.assertIn("= NaN\n", self.getFormatted("float", "-1"))<br class="">
+ # Checks the float16 code.<br class="">
+ self.assertIn("= 2\n", self.getFormatted("float", "(__UINT16_TYPE__)0x4000"))<br class="">
+ self.assertIn("= error: unsupported byte size (1) for float format\n", self.getFormatted("float", "'a'"))<br class="">
<br class="">
# enumeration<br class="">
self.assertIn("= 0\n", self.getFormatted("enumeration", "0"))<br class="">
@@ -59,6 +62,13 @@ def test(self):<br class="">
<br class="">
# octal<br class="">
self.assertIn("= 04553207\n", self.getFormatted("octal", "1234567"))<br class="">
+ self.assertIn("= 0221505317046536757\n", self.getFormatted("octal", "(__uint128_t)0x123456789ABDEFull"))<br class="">
+<br class="">
+ # complex float<br class="">
+ self.assertIn("= error: unsupported byte size (1) for complex float format\n", self.getFormatted("complex float", "'a'"))<br class="">
+<br class="">
+ # complex integer<br class="">
+ self.assertIn("= error: unsupported byte size (1) for complex integer format\n", self.getFormatted("complex integer", "'a'"))<br class="">
<br class="">
# hex<br class="">
self.assertIn("= 0x00abc123\n", self.getFormatted("hex", "0xABC123"))<br class="">
@@ -86,6 +96,17 @@ def test(self):<br class="">
self.assertIn('= " \\U0000001b\\a\\b\\f\\n\\r\\t\\vaA09"\n', self.getFormatted("OSType", "cstring"))<br class="">
self.assertIn('= " \\U0000001b\\a\\b\\f\\n\\r\\t\\vaA09"\n', self.getFormatted("unicode8", "cstring"))<br class="">
<br class="">
+ # FIXME: Passing a 'const char *' will ignore any given format,<br class="">
+ # so we have to repeat the tests with a void* casts to actually test our formats.<br class="">
+ self.assertIn('= \\x9a\\x0f\\0\\0\\x01\\0\\0\\0\n', self.getFormatted("character array", "(void *)cstring"))<br class="">
+ self.assertIn('= \\x9a\\x0f\\0\\0\\x01\\0\\0\\0\n', self.getFormatted("character", "(void *)cstring"))<br class="">
+ self.assertIn('= " \\e\\a\\b\\f\\n\\r\\t\\vaA09"\n', self.getFormatted("c-string", "(void *)cstring"))<br class="">
+ # FIXME: Ignores the printables characters at the end.<br class="">
+ self.assertIn('= ........\n', self.getFormatted("printable character", "(void *)cstring"))<br class="">
+ self.assertIn('= \'\\0\\0\\0\\x01\\0\\0\\x0f\\x9a\'\n', self.getFormatted("OSType", "(void *)cstring"))<br class="">
+ # FIXME: This should print a string.<br class="">
+ self.assertIn('= 0x0000000100000f9a\n', self.getFormatted("unicode8", "(void *)cstring"))<br class="">
+<br class="">
self.assertIn('= \\0\\0\\0\\0\\0\\0\\0\\0\n', self.getFormatted("character array", "(__UINT64_TYPE__)0"))<br class="">
self.assertIn('= \\0\\0\\0\\0\\0\\0\\0\\0\n', self.getFormatted("character", "(__UINT64_TYPE__)0"))<br class="">
self.assertIn('=\n', self.getFormatted("c-string", "(__UINT64_TYPE__)0"))<br class="">
@@ -137,6 +158,11 @@ def test(self):<br class="">
# There is not int128_t[] style, so this only tests uint128_t[].<br class="">
self.assertIn("= {0x00000000000000000123456789abcdef}\n", self.getFormatted("uint128_t[]", "__uint128_t i = 0x123456789ABCDEF; i"))<br class="">
<br class="">
+ # Different fixed-width float type arrays.<br class="">
+ self.assertIn("{2 2}\n", self.getFormatted("float16[]", "0x40004000"))<br class="">
+ self.assertIn("{2 2}\n", self.getFormatted("float32[]", "0x4000000040000000ll"))<br class="">
+ self.assertIn("{2 0}\n", self.getFormatted("float64[]", "__uint128_t i = 0x4000000000000000ll; i"))<br class="">
+<br class="">
# Invalid format string<br class="">
self.expect("expr --format invalid_format_string -- 1", error=True,<br class="">
substrs=["error: Invalid format character or name 'invalid_format_string'. Valid values are:"])<br class="">
@@ -144,10 +170,12 @@ def test(self):<br class="">
# Extends to host target pointer width.<br class="">
@skipIf(archs=no_match(['x86_64']))<br class="">
@no_debug_info_test<br class="">
- def test_instruction(self):<br class="">
+ def test_pointer(self):<br class="">
# pointer<br class="">
self.assertIn("= 0x000000000012d687\n", self.getFormatted("pointer", "1234567"))<br class="">
self.assertIn("= 0x0000000000000000\n", self.getFormatted("pointer", "0"))<br class="">
+ # FIXME: Just ignores the input value as it's not pointer sized.<br class="">
+ self.assertIn("= 0x0000000000000000\n", self.getFormatted("pointer", "'a'"))<br class="">
<br class="">
# Depends on the host target for decoding.<br class="">
@skipIf(archs=no_match(['x86_64']))<br class="">
<br class="">
<br class="">
<br class="">
_______________________________________________<br class="">
lldb-commits mailing list<br class="">
<a href="mailto:lldb-commits@lists.llvm.org" target="_blank" class="">lldb-commits@lists.llvm.org</a><br class="">
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank" class="">https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br class="">
</blockquote></div>
</div></blockquote></div><br class=""></div></body></html>