[all-commits] [llvm/llvm-project] e9b11a: [ASan] Make duplicate_os_log_reports.cpp work with...

Aiden Grossman via All-commits all-commits at lists.llvm.org
Wed Nov 19 09:26:52 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e9b11ae83731bbca6adf9f9b39b6d3aa1869e445
      https://github.com/llvm/llvm-project/commit/e9b11ae83731bbca6adf9f9b39b6d3aa1869e445
  Author: Aiden Grossman <aidengrossman at google.com>
  Date:   2025-11-19 (Wed, 19 Nov 2025)

  Changed paths:
    A compiler-rt/test/asan/TestCases/Darwin/Inputs/check-syslog.sh
    M compiler-rt/test/asan/TestCases/Darwin/duplicate_os_log_reports.cpp

  Log Message:
  -----------
  [ASan] Make duplicate_os_log_reports.cpp work with the internal shell

This test used a for loop to implement retries and also did some trickery with PIDs.
For this test, just invoke bash for actually running the test given we need the PID,
and move the for loop into a separate shell script file that we can then invoke from
within the test. Normally it would make sense to rewrite such a script in Python, but
given this test does not have portability concerns only running on Darwin, it is fine
to use a shell script here given there is no other convenient alternative.

Reviewers: ndrewh, DanBlackwell, fmayer

Reviewed By: ndrewh

Pull Request: https://github.com/llvm/llvm-project/pull/168656



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list