[all-commits] [llvm/llvm-project] cde866: Various minor fixes for python 3

Sterling-Augustine via All-commits all-commits at lists.llvm.org
Tue Jun 8 16:11:48 PDT 2021


  Branch: refs/heads/release/12.x
  Home:   https://github.com/llvm/llvm-project
  Commit: cde86632a772b523ba3db7039f75d979f557b57c
      https://github.com/llvm/llvm-project/commit/cde86632a772b523ba3db7039f75d979f557b57c
  Author: Sterling Augustine <saugustine at google.com>
  Date:   2021-06-08 (Tue, 08 Jun 2021)

  Changed paths:
    M libcxx/test/libcxx/gdb/gdb_pretty_printer_test.sh.cpp
    M libcxx/utils/gdb/libcxx/printers.py

  Log Message:
  -----------
  Various minor fixes for python 3

Switch StdTuple printer from python 2-style "next" to python 3.

Nested iteration changed enough to make the original bitset iteration
code a bit trickier than it needs to be, so unnest.

The end node of a map iterator is sometimes hard to detect in isolation,
don't fail in that case.

Differential Revision: https://reviews.llvm.org/D96167

(cherry picked from commit a34b8b879e345397880c1f9f8de4c294dd0b370c)


  Commit: cf3e126e6449255ef183df71f5c54020d7a87ee5
      https://github.com/llvm/llvm-project/commit/cf3e126e6449255ef183df71f5c54020d7a87ee5
  Author: Petr Hosek <phosek at google.com>
  Date:   2021-06-08 (Tue, 08 Jun 2021)

  Changed paths:
    M libcxx/test/libcxx/gdb/gdb_pretty_printer_test.py

  Log Message:
  -----------
  [libcxx] Make the GDB pretty printer test less strict

This is a workaround for PR48937. GDB can sometimes print additional
warnings which currently fails the test. Use re.search instead of
re.match to ignore this additional output.

Differential Revision: https://reviews.llvm.org/D99532

(cherry picked from commit 9ac988f6a80aa1dd25594a8b4c86c0380ac99466)


  Commit: bf25180e6727f53b916c73e53212b274fe64ded6
      https://github.com/llvm/llvm-project/commit/bf25180e6727f53b916c73e53212b274fe64ded6
  Author: Sterling Augustine <saugustine at google.com>
  Date:   2021-06-08 (Tue, 08 Jun 2021)

  Changed paths:
    M libcxx/test/libcxx/gdb/gdb_pretty_printer_test.py
    M libcxx/test/libcxx/gdb/gdb_pretty_printer_test.sh.cpp
    M libcxx/utils/gdb/libcxx/printers.py

  Log Message:
  -----------
  Tolerate missing debug info in the shared_ptr pretty printer.

Certain fields of shared ptr have virtual functions and therefore
have their debug info homed in libc++. But if libc++ wasn't built
with debug info, the pretty printer would fail.

This patch makes the pretty printer tolerate such conditions and
updates the test harness.

This patch significantly reworks a previous attempt.

This addresses https://bugs.llvm.org/show_bug.cgi?id=48937

Differential Revision: https://reviews.llvm.org/D100610

(cherry picked from commit 55b7061116b5f0f839bd4240c3c6fba63918b816)


  Commit: 1539c543dbe5edb9d6809aa8ad60b32dba5888d7
      https://github.com/llvm/llvm-project/commit/1539c543dbe5edb9d6809aa8ad60b32dba5888d7
  Author: Sterling Augustine <saugustine at google.com>
  Date:   2021-06-08 (Tue, 08 Jun 2021)

  Changed paths:
    M libcxx/test/libcxx/gdb/gdb_pretty_printer_test.py

  Log Message:
  -----------
  Don't fail the shared_ptr test if libc++ has insufficient debug info.

Don't fail the shared_ptr test if libc++ has insufficient debug info.

This addresses https://bugs.llvm.org/show_bug.cgi?id=48937

Differential Revision: https://reviews.llvm.org/D100610

(cherry picked from commit 0e8378032597bcaccb948de88e965ad75bfaeb7b)


Compare: https://github.com/llvm/llvm-project/compare/0826268d59c6...1539c543dbe5


More information about the All-commits mailing list