jsjodin added a comment.

In D142244#4096777 <https://reviews.llvm.org/D142244#4096777>, @clementval 
wrote:

> In D142244#4096648 <https://reviews.llvm.org/D142244#4096648>, @jsjodin wrote:
>
>> In D142244#4096579 <https://reviews.llvm.org/D142244#4096579>, @clementval 
>> wrote:
>>
>>> In D142244#4096546 <https://reviews.llvm.org/D142244#4096546>, @jsjodin 
>>> wrote:
>>>
>>>> In D142244#4096538 <https://reviews.llvm.org/D142244#4096538>, @clementval 
>>>> wrote:
>>>>
>>>>> In D142244#4096512 <https://reviews.llvm.org/D142244#4096512>, @jsjodin 
>>>>> wrote:
>>>>>
>>>>>> No, it should work for any configuration as far as I know. How are you 
>>>>>> running the test?
>>>>>
>>>>> Just with `check-flang`
>>>>
>>>> How does it fail? Can you show the output? Also what CPU are you running 
>>>> on?
>>>
>>>
>>>
>>>   I run on `x86_64`
>>
>> How did you configure your build? I'd like to reproduce your build and see 
>> if I can make it fail on my machine. Right now the test passes.
>
> My bad .. my `CompilerInvocation.cpp` was overwritten and missed part of this 
> patch. It works fine. Sorry to have bothered you with that.

No problem, glad it works for you. I'm working on another patch that is 
configuration dependent so I want to make sure there are no errors.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142244/new/

https://reviews.llvm.org/D142244

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to