[PATCH] Improve big-endian support for ARM and AArch64

Renato Golin renato.golin at linaro.org
Wed Sep 24 15:27:28 PDT 2014


Hi Charlie,

I reverted the changes, since the patches have shown many failures on
Windows, i686 and hexagon.


** First, the processes-relocations.ll file shouldn't have all archs,
since some bots only build one or two back-ends:

http://lab.llvm.org:8011/builders/llvm-hexagon-elf/builds/11354/steps/test-llvm/logs/LLVM%3A%3Aprocesses-relocations.ll

When I fixed that, I learnt that they don't support that kind of file either:

http://lab.llvm.org:8011/builders/llvm-hexagon-elf/builds/11362/steps/test-llvm/logs/LLVM%3A%3Aprocesses-relocations.ll

So, an XFAIL on hexagon is probably a good idea.


** Second, Windows machines failed, since they don't support Dwarf:

http://bb.pgr.jp/builders/ninja-x64-msvc-RA-centos6/builds/6585

Eric hinted to look at nearby tests and make sure to follow the same pattern.


** Third, i686 has an error::

http://bb.pgr.jp/builders/clang-i686-cygwin-RA-centos6/builds/10472/steps/test_llvmclang/logs/LLVM%20%3A%3A%20DebugInfo__processes-relocations.ll

Though, i386 works, at least on FreeBSD (where XFAIL i386 actually passes):

http://llvm-amd64.freebsd.your.org/b/builders/clang-i386-freebsd/builds/23659/steps/check-all.1/logs/stdio

A better alternative is to split that test into multiple files, one on
each arch's directory (at least ARM, AArch64 and x86_64).


I have reverted all patches for now, since they were causing a lot of
trouble. In the future, after you can get all those fixes in and
validate, we should commit one by one and wait for the dust to settle.

cheers,
--renato



More information about the llvm-commits mailing list