arphaman added a comment.

In D65907#1643800 <https://reviews.llvm.org/D65907#1643800>, @JamesNagurne 
wrote:

> In D65907#1643650 <https://reviews.llvm.org/D65907#1643650>, @arphaman wrote:
>
> > No the windows test failure was different, there were no Deps at all. I'm 
> > currently investigating it on a windows VM.
> >
> > @JamesNagurne I think there's some issue with the working directory, which 
> > is not added in your case. Which platform are you running your build/test 
> > on? Which cmake options are you using?
>
>
> I apologize for not giving such information in the first reply. Unfortunately 
> this isn't an easy remote reproduction, as our ToolChain and some integral 
> changes aren't upstreamed. This is an embedded ARM cross-compiled on Linux. 
> Might be able to reproduce with arm-none-none-eabi.
>  LLVM is built as an external project. Looking at the build system, it looks 
> like we have the CMAKE_ARGS:
>
>  
>   -DLLVM_DEFAULT_TARGET_TRIPLE=arm-ti-none-eabi
>   -DLLVM_EXTERNAL_CLANG_SOURCE_DIR=${CMAKE_SOURCE_DIR}/llvm-project/clang
>   -DLLVM_TARGETS_TO_BUILD=ARM
>   -DCMAKE_BUILD_TYPE=Release
>   -DCMAKE_CXX_COMPILER=clang++
>   -DCMAKE_C_COMPILER=clang
>   -DLLVM_USE_LINKER=gold
>   -GNinja
>   
>
> Nothing suspicious, except maybe the default triple and ARM target.
>  Looking at our (not upstream) toolchain file, we do have some RTLibs, 
> LibInternal, libcxx, and System include changes, along with a -nostdsysteminc 
> to avoid pulling host includes into our cross compiler. However, none of this 
> should affect general "#include" behavior, correct?
>  Another glance at your changes don't seem to affect any target-specific 
> handling either, at least directly.


Yes, I don't see anything in your changes that is an obvious thing to blame.

> I might have to just bite the bullet and drop into the test with a debugger.

I fixed the Windows issue, and it was completely unrelated to your issue. It 
looks like the FileManager isn't constructing absolute paths when accessing the 
files, which is somewhat unexpected. It would be useful to debug invocations of 
`getFileEntryRef`, to see if the `InterndFilename` in the function is getting 
changed into an absolute path or not.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65907



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

Reply via email to