[clang] [llvm] [llvm][Support] Stop using PWD in current_path (PR #94544)

IƱaki Amatria Barral via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 13 06:27:35 PDT 2024


================
@@ -17,10 +17,6 @@
 // GCNO-LOCATION: "-coverage-notes-file={{.*}}/foo/bar.gcno"
 // GCNO-LOCATION-REL: "-coverage-notes-file={{.*}}{{/|\\\\}}foo/bar.gcno"
 
-/// GCC allows PWD to change the paths.
-// RUN: %if system-linux %{ PWD=/proc/self/cwd %clang -### -c --coverage %s -o foo/bar.o 2>&1 | FileCheck --check-prefix=PWD %s %}
-// PWD: "-coverage-notes-file=/proc/self/cwd/foo/bar.gcno" "-coverage-data-file=/proc/self/cwd/foo/bar.gcda"
----------------
inaki-amatria wrote:

Thank you for the timely response and for raising this important concern.

I understand the importance of maintaining consistent behavior in Clang, and I certainly do not wish to introduce changes that might silently break existing build scripts.

Could you please guide me on how to initiate the RFC process? I would like to ensure that we address this issue appropriately.

Thank you again for your insights, and I look forward to your guidance on progressing with the RFC.

https://github.com/llvm/llvm-project/pull/94544


More information about the llvm-commits mailing list