Skip to content

[llvm-lit] Lit Internal Shell -Fix Command Not Found Error #102200

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

Harini0924
Copy link
Contributor

Added env in front of LLVM_PROFILE_FILE to properly set the environment
variable correctly.

Added `env` in front of `LLVM_PROFILE_FILE` to properly set the environment
variable correctly.
@llvmbot llvmbot added compiler-rt PGO Profile Guided Optimizations labels Aug 6, 2024
@Harini0924
Copy link
Contributor Author

CC: @ilovepi @petrhosek

@llvmbot
Copy link
Member

llvmbot commented Aug 6, 2024

@llvm/pr-subscribers-pgo

Author: None (Harini0924)

Changes

Added env in front of LLVM_PROFILE_FILE to properly set the environment
variable correctly.


Full diff: https://github.com/llvm/llvm-project/pull/102200.diff

1 Files Affected:

  • (modified) compiler-rt/test/profile/Linux/instrprof-comdat.test (+1-1)
diff --git a/compiler-rt/test/profile/Linux/instrprof-comdat.test b/compiler-rt/test/profile/Linux/instrprof-comdat.test
index cbeeb91a64a4c..d71b49f6e06c7 100644
--- a/compiler-rt/test/profile/Linux/instrprof-comdat.test
+++ b/compiler-rt/test/profile/Linux/instrprof-comdat.test
@@ -1,6 +1,6 @@
 RUN: mkdir -p %t.d
 RUN: %clangxx_profgen -o %t.d/comdat -fcoverage-mapping -fuse-ld=gold %S/../Inputs/instrprof-comdat-1.cpp %S/../Inputs/instrprof-comdat-2.cpp
-RUN: LLVM_PROFILE_FILE=%t-comdat.profraw %run %t.d/comdat
+RUN: env LLVM_PROFILE_FILE=%t-comdat.profraw %run %t.d/comdat
 RUN: llvm-profdata merge -o %t.d/comdat.prof %t-comdat.profraw 
 RUN: llvm-cov show --instr-profile=%t.d/comdat.prof %t.d/comdat | FileCheck --check-prefix=HEADER %S/../Inputs/instrprof-comdat.h
 

@Harini0924
Copy link
Contributor Author

I will add this change to my existing PR: #102069

@Harini0924 Harini0924 closed this Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compiler-rt PGO Profile Guided Optimizations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants