[all-commits] [llvm/llvm-project] 5b614e: [lldb] Let dotest check for the `cassert` header i...

Raphael Isemann via All-commits all-commits at lists.llvm.org
Mon May 17 02:17:37 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5b614eb4ea6ce047daebc0517eb4efa83123fe58
      https://github.com/llvm/llvm-project/commit/5b614eb4ea6ce047daebc0517eb4efa83123fe58
  Author: Raphael Isemann <teemperor at gmail.com>
  Date:   2021-05-17 (Mon, 17 May 2021)

  Changed paths:
    M lldb/packages/Python/lldbsuite/test/dotest.py

  Log Message:
  -----------
  [lldb] Let dotest check for the `cassert` header instead of `algorithm` for libc++ check

This is just a dotest check to see if we can compile a simple program that uses
libc++. Right now we are parsing the rather big `algorithm` header in the test
program, but the test really just checks whether we can find *any* libc++
headers and link against some libc++ SO. Using the much smaller `cassert` header
for checking whether we can find libc++ headers speeds up this check by a bit.

After some incredibly unscientific performance testing this saves a few seconds
when running the test suite on Linux (on macOS we hardcoded that libc++ is
always there, so this check won't be used there and we don't save any time).

Reviewed By: jankratochvil

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




More information about the All-commits mailing list