[compiler-rt] r264773 - [profile] Make a test work if run by the super-user

Vedant Kumar via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 29 13:09:09 PDT 2016


> On Mar 29, 2016, at 1:07 PM, Hal Finkel <hfinkel at anl.gov> wrote:
> 
> ----- Original Message -----
>> From: "Vedant Kumar via llvm-commits" <llvm-commits at lists.llvm.org>
>> To: "Sean Silva" <chisophugis at gmail.com>
>> Cc: "llvm-commits" <llvm-commits at lists.llvm.org>
>> Sent: Tuesday, March 29, 2016 2:55:32 PM
>> Subject: Re: [compiler-rt] r264773 - [profile] Make a test work if run by the super-user
>> 
>> Darwin is funny. I tried an empty string also, but the test failed
>> :(.
>> 
>> That said, it's important the test is portable. I'll revert and try
>> again if the Windows bots have issues.
> 
> Shouldn't LLVM_PROFILE_FILE= give the default file name?
> 
> Maybe use something like: /doesnotexist?

I think the runtime would then create /doesnotexist, so the test would still fail.

vedant


> -Hal
> 
>> 
>> vedant
>> 
>>> On Mar 29, 2016, at 12:52 PM, Sean Silva <chisophugis at gmail.com>
>>> wrote:
>>> 
>>> An empty string also works for me and is probably more portable
>>> (i.e. `env LLVM_PROFILE_FILE= LLVM_PROFILE_VERBOSE_ERRORS=1 %run
>>> ...`). Does that work for you?
>>> 
>>> -- Sean Silva
>>> 
>>> On Tue, Mar 29, 2016 at 12:24 PM, Vedant Kumar via llvm-commits
>>> <llvm-commits at lists.llvm.org> wrote:
>>> Author: vedantk
>>> Date: Tue Mar 29 14:24:58 2016
>>> New Revision: 264773
>>> 
>>> URL: http://llvm.org/viewvc/llvm-project?rev=264773&view=rev
>>> Log:
>>> [profile] Make a test work if run by the super-user
>>> 
>>> Modified:
>>>    compiler-rt/trunk/test/profile/instrprof-error.c
>>> 
>>> Modified: compiler-rt/trunk/test/profile/instrprof-error.c
>>> URL:
>>> http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/profile/instrprof-error.c?rev=264773&r1=264772&r2=264773&view=diff
>>> ==============================================================================
>>> --- compiler-rt/trunk/test/profile/instrprof-error.c (original)
>>> +++ compiler-rt/trunk/test/profile/instrprof-error.c Tue Mar 29
>>> 14:24:58 2016
>>> @@ -1,8 +1,5 @@
>>> // RUN: %clang_profgen -o %t -O3 %s
>>> -// RUN: touch %t.profraw
>>> -// RUN: chmod -w %t.profraw
>>> -// RUN: env LLVM_PROFILE_FILE=%t.profraw
>>> LLVM_PROFILE_VERBOSE_ERRORS=1 %run %t 1 2>&1 | FileCheck %s
>>> -// RUN: chmod +w %t.profraw
>>> +// RUN: env LLVM_PROFILE_FILE="/" LLVM_PROFILE_VERBOSE_ERRORS=1
>>> %run %t 1 2>&1 | FileCheck %s
>>> 
>>> int main(int argc, const char *argv[]) {
>>>   if (argc < 2)
>>> 
>>> 
>>> _______________________________________________
>>> llvm-commits mailing list
>>> llvm-commits at lists.llvm.org
>>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>>> 
>> 
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>> 
> 
> -- 
> Hal Finkel
> Assistant Computational Scientist
> Leadership Computing Facility
> Argonne National Laboratory



More information about the llvm-commits mailing list