[clang] [clang][test] Rewrote test to work with lit internal shell syntax (PR #105902)

2024-08-26 Thread Connie Zhu via cfe-commits
https://github.com/connieyzhu updated https://github.com/llvm/llvm-project/pull/105902 >From 5801e58f5e89fb90a3b18414e1cb959d027b4fee Mon Sep 17 00:00:00 2001 From: Connie Zhu Date: Fri, 23 Aug 2024 17:23:22 + Subject: [PATCH 1/2] [clang][test] Rewrote test to work with lit internal shell

[clang] [clang][test] Rewrote test to work with lit internal shell syntax (PR #105902)

2024-08-26 Thread Connie Zhu via cfe-commits
@@ -0,0 +1,22 @@ +import argparse +import os + +def get_file_size(file_path): +try: +return os.path.getsize(file_path) +except: +print(f"Unable to get file size of {file_path}") +return None connieyzhu wrote: Now that I look at it

[clang] [clang][test] Rewrote test to work with lit internal shell syntax (PR #105902)

2024-08-26 Thread Connie Zhu via cfe-commits
https://github.com/connieyzhu updated https://github.com/llvm/llvm-project/pull/105902 >From 5801e58f5e89fb90a3b18414e1cb959d027b4fee Mon Sep 17 00:00:00 2001 From: Connie Zhu Date: Fri, 23 Aug 2024 17:23:22 + Subject: [PATCH 1/3] [clang][test] Rewrote test to work with lit internal shell

[clang] [clang][test] Rewrote test using $() to work with lit internal shell syntax (PR #105902)

2024-08-26 Thread Connie Zhu via cfe-commits
https://github.com/connieyzhu edited https://github.com/llvm/llvm-project/pull/105902 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [compiler-rt] [clang][compiler-rt][test] Removed dirname command substitutions from tests (PR #105754)

2024-08-26 Thread Connie Zhu via cfe-commits
https://github.com/connieyzhu closed https://github.com/llvm/llvm-project/pull/105754 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][test] Rewrote test using $() to work with lit internal shell syntax (PR #105902)

2024-08-28 Thread Connie Zhu via cfe-commits
https://github.com/connieyzhu updated https://github.com/llvm/llvm-project/pull/105902 >From 5801e58f5e89fb90a3b18414e1cb959d027b4fee Mon Sep 17 00:00:00 2001 From: Connie Zhu Date: Fri, 23 Aug 2024 17:23:22 + Subject: [PATCH 1/4] [clang][test] Rewrote test to work with lit internal shell

[clang] [clang][test] Rewrote test using $() to work with lit internal shell syntax (PR #105902)

2024-08-28 Thread Connie Zhu via cfe-commits
https://github.com/connieyzhu updated https://github.com/llvm/llvm-project/pull/105902 >From 5801e58f5e89fb90a3b18414e1cb959d027b4fee Mon Sep 17 00:00:00 2001 From: Connie Zhu Date: Fri, 23 Aug 2024 17:23:22 + Subject: [PATCH 1/5] [clang][test] Rewrote test to work with lit internal shell

[clang] [clang][test] Rewrote test using command substitution to work with lit internal shell syntax (PR #105902)

2024-08-29 Thread Connie Zhu via cfe-commits
https://github.com/connieyzhu edited https://github.com/llvm/llvm-project/pull/105902 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][test] Rewrote test using command substitution to work with lit internal shell syntax (PR #105902)

2024-08-29 Thread Connie Zhu via cfe-commits
https://github.com/connieyzhu closed https://github.com/llvm/llvm-project/pull/105902 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [compiler-rt] [clang][compiler-rt][test] Removed dirname command substitutions from tests (PR #105754)

2024-08-22 Thread Connie Zhu via cfe-commits
https://github.com/connieyzhu created https://github.com/llvm/llvm-project/pull/105754 This patch rewrites tests in clang and compiler-rt that uses bash command substitution syntax $() to execute the dirname command. This is done so that the tests can be run using lit's internal shell. Fixes

[clang] [clang][test] Rewrote test to work with lit internal shell syntax (PR #105902)

2024-08-23 Thread Connie Zhu via cfe-commits
https://github.com/connieyzhu created https://github.com/llvm/llvm-project/pull/105902 This patch rewrites a test that uses command substitution `$()` and the `stat` command, which are not supported by lit's internal shell. Instead of using this syntax to perform the file size comparison done

[clang] [clang][test] Rewrote test to work with lit internal shell syntax (PR #105902)

2024-08-23 Thread Connie Zhu via cfe-commits
https://github.com/connieyzhu updated https://github.com/llvm/llvm-project/pull/105902 >From b7cf36a247e480ddefc1708cb68078b440011d63 Mon Sep 17 00:00:00 2001 From: Connie Zhu Date: Fri, 23 Aug 2024 17:23:22 + Subject: [PATCH] [clang][test] Rewrote test to work with lit internal shell synt

[clang] [compiler-rt] [clang][compiler-rt][test] Removed dirname command substitutions from tests (PR #105754)

2024-08-23 Thread Connie Zhu via cfe-commits
https://github.com/connieyzhu updated https://github.com/llvm/llvm-project/pull/105754 >From cfd90f666e9b6f65e277bc753de973a748a7b275 Mon Sep 17 00:00:00 2001 From: Connie Zhu Date: Thu, 22 Aug 2024 23:32:40 + Subject: [PATCH 1/2] [clang][compiler-rt][test] Removed dirname command substitu

[clang] [compiler-rt] [clang][compiler-rt][test] Removed dirname command substitutions from tests (PR #105754)

2024-08-23 Thread Connie Zhu via cfe-commits
@@ -1,8 +1,9 @@ -// RUN: %clang_hwasan -Wl,--build-id -g %s -o %t -// RUN: echo '[{"prefix": "'"$(realpath $(dirname %s))"'/", "link": "http://test.invalid/{file}:{line}"}]' > %t.linkify -// RUN: %env_hwasan_opts=symbolize=0 not %run %t 2>&1 | hwasan_symbolize --html --symbols $

[clang] [compiler-rt] [clang][compiler-rt][test] Removed dirname command substitutions from tests (PR #105754)

2024-08-23 Thread Connie Zhu via cfe-commits
https://github.com/connieyzhu updated https://github.com/llvm/llvm-project/pull/105754 >From 52addf5d3e0e4752d9a1e6f5145dc37e9132bbe5 Mon Sep 17 00:00:00 2001 From: Connie Zhu Date: Thu, 22 Aug 2024 23:32:40 + Subject: [PATCH 1/3] [clang][compiler-rt][test] Removed dirname command substitu