[all-commits] [llvm/llvm-project] 37ec83: [lldb] Use file to synchronize TestDeepBundle and ...

Jonas Devlieghere via All-commits all-commits at lists.llvm.org
Fri Aug 14 08:33:09 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 37ec83fcfc6c7915c51268f578b8e0dadb54c1cf
      https://github.com/llvm/llvm-project/commit/37ec83fcfc6c7915c51268f578b8e0dadb54c1cf
  Author: Jonas Devlieghere <jonas at devlieghere.com>
  Date:   2020-08-14 (Fri, 14 Aug 2020)

  Changed paths:
    M lldb/test/API/macosx/find-dsym/bundle-with-dot-in-filename/TestBundleWithDotInFilename.py
    M lldb/test/API/macosx/find-dsym/bundle-with-dot-in-filename/main.c
    M lldb/test/API/macosx/find-dsym/deep-bundle/TestDeepBundle.py
    M lldb/test/API/macosx/find-dsym/deep-bundle/main.c

  Log Message:
  -----------
  [lldb] Use file to synchronize TestDeepBundle and TestBundleWithDotInFilename

Currently these two tests use an arbitrary wait of 5 seconds for the
inferior to finish setting up. When the test machine is under heavy load
this sometimes is insufficient leading to spurious test failures. This
patch adds synchronization trough a token on the file system. In
addition to making the test more reliable it also makes it much faster
because we no longer have to wait the full 5 seconds if the setup was
completed faster than that.

Differential revision: https://reviews.llvm.org/D85915




More information about the All-commits mailing list