[clang] [clang][modules] Strip LLVM options (PR #75405)

https://github.com/ributzka closed https://github.com/llvm/llvm-project/pull/75405 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][modules] Strip LLVM options (PR #75405)

ributzka wrote: Thanks for the reviews. https://github.com/llvm/llvm-project/pull/75405 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][modules] Strip LLVM options (PR #75405)

https://github.com/ributzka updated https://github.com/llvm/llvm-project/pull/75405 >From 78d332143efc8220d80715091246da4e62fdfb14 Mon Sep 17 00:00:00 2001 From: Juergen Ributzka Date: Wed, 13 Dec 2023 15:44:54 -0800 Subject: [PATCH] [clang][modules] Strip LLVM options Currently, the dep scann

[clang] [clang][modules] Strip LLVM options (PR #75405)

https://github.com/Bigcheese approved this pull request. https://github.com/llvm/llvm-project/pull/75405 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][modules] Strip LLVM options (PR #75405)

https://github.com/jansvoboda11 approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/75405 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][modules] Strip LLVM options (PR #75405)

@@ -0,0 +1,49 @@ +// RUN: rm -rf %t +// RUN: split-file %s %t +// RUN: sed -e "s|DIR|%/t|g" %t/cdb1.json.template > %t/cdb1.json + +// RUN: clang-scan-deps -compilation-database %t/cdb1.json -format experimental-full > %t/result1.txt +// RUN: FileCheck %s -input-file %t/result1.t

[clang] [clang][modules] Strip LLVM options (PR #75405)

@@ -0,0 +1,49 @@ +// RUN: rm -rf %t +// RUN: split-file %s %t +// RUN: sed -e "s|DIR|%/t|g" %t/cdb1.json.template > %t/cdb1.json + +// RUN: clang-scan-deps -compilation-database %t/cdb1.json -format experimental-full > %t/result1.txt +// RUN: FileCheck %s -input-file %t/result1.t

[clang] [clang][modules] Strip LLVM options (PR #75405)

https://github.com/ributzka updated https://github.com/llvm/llvm-project/pull/75405 >From ef0b260488d1de1cc89ead90d35cc6abf189fd6c Mon Sep 17 00:00:00 2001 From: Juergen Ributzka Date: Wed, 13 Dec 2023 15:44:54 -0800 Subject: [PATCH 1/2] [clang][modules] Strip LLVM options Currently, the dep s

[clang] [clang][modules] Strip LLVM options (PR #75405)

@@ -0,0 +1,49 @@ +// RUN: rm -rf %t +// RUN: split-file %s %t +// RUN: sed -e "s|DIR|%/t|g" %t/cdb1.json.template > %t/cdb1.json + +// RUN: clang-scan-deps -compilation-database %t/cdb1.json -format experimental-full > %t/result1.txt +// RUN: FileCheck %s -input-file %t/result1.t

[clang] [clang][modules] Strip LLVM options (PR #75405)

@@ -0,0 +1,49 @@ +// RUN: rm -rf %t +// RUN: split-file %s %t +// RUN: sed -e "s|DIR|%/t|g" %t/cdb1.json.template > %t/cdb1.json + +// RUN: clang-scan-deps -compilation-database %t/cdb1.json -format experimental-full > %t/result1.txt +// RUN: FileCheck %s -input-file %t/result1.t

[clang] [clang][modules] Strip LLVM options (PR #75405)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Juergen Ributzka (ributzka) Changes Currently, the dep scanner does not remove LLVM options from the argument list. Since LLVM options shouldn't affect the AST, it is safe to remove them all. --- Full diff: https://github.com/llvm/llvm-pr

[clang] [clang][modules] Strip LLVM options (PR #75405)

https://github.com/ributzka created https://github.com/llvm/llvm-project/pull/75405 Currently, the dep scanner does not remove LLVM options from the argument list. Since LLVM options shouldn't affect the AST, it is safe to remove them all. >From ef0b260488d1de1cc89ead90d35cc6abf189fd6c Mon Sep