<div dir="ltr">Hi Renato,<div><br></div><div>Nothing in particular springs to mind. Is it possible that in the context where this part of the Makefile is running Makefile.config isn't available?</div><div><br></div><div>
I'm not sure how this can be completely broken in the fashion you describe though, as it seems like that would make little-endian fail which is what most people are testing.</div><div><br></div><div> - Daniel</div></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Feb 20, 2014 at 7:14 AM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Daniel,<br>
<br>
I know you only did a small change to support big/little endian<br>
reference outputs, but maybe you can help me.<br>
<br>
I'm running the test-suite on AArch64 and it's correctly detecting<br>
little-endian, even setting the ENDIAN=little on configure and<br>
Makefiles alike, but it still generates "big-endian" from<br>
Makefile.programs.<br>
<br>
Here's the first lines of:<br>
<br>
sandbox/test-...$ grep -i endian * | less<br>
config.log:|  not big endian<br>
config.log:ac_cv_c_bigendian=no<br>
config.log:ENDIAN='little'<br>
config.status:s,@ENDIAN@,|#_!!_#|little,g<br>
configure.log:checking whether byte ordering is bigendian... no<br>
grep: External: Is a directory<br>
Makefile.config:ENDIAN := little<br>
<br>
=== So far, so good.<br>
<br>
<br>
Makefile.programs:      -if [ -f<br>
"$(PROJ_SRC_DIR)/$*.reference_output.$(ENDIAN)-endian.$(REFERENCE_OUTPUT_KEY)"<br>
]; then \<br>
Makefile.programs:        cp<br>
$(PROJ_SRC_DIR)/$*.reference_output.$(ENDIAN)-endian.$(REFERENCE_OUTPUT_KEY)<br>
$@; \<br>
Makefile.programs:      elif [ -f<br>
"$(PROJ_SRC_DIR)/$*.reference_output.$(ENDIAN)-endian" ]; then \<br>
Makefile.programs:        cp<br>
$(PROJ_SRC_DIR)/$*.reference_output.$(ENDIAN)-endian $@; \<br>
<br>
=== This is correct, and I was assuming ENDIAN := little, as said above.<br>
=== But then you get this on ALL tests in test.log:<br>
<br>
test.log:if [ -f<br>
"/home/enevill/llvm/test/test-suite/SingleSource/Regression/C++/EH/ConditionalExpr.reference_output.big-endian."<br>
]; then \<br>
test.log:         cp<br>
/home/enevill/llvm/test/test-suite/SingleSource/Regression/C++/EH/ConditionalExpr.reference_output.big-endian.<br>
Output/ConditionalExpr.out-nat; \<br>
test.log:       elif [ -f<br>
"/home/enevill/llvm/test/test-suite/SingleSource/Regression/C++/EH/ConditionalExpr.reference_output.big-endian"<br>
]; then \<br>
test.log:         cp<br>
/home/enevill/llvm/test/test-suite/SingleSource/Regression/C++/EH/ConditionalExpr.reference_output.big-endian<br>
Output/ConditionalExpr.out-nat; \<br>
<br>
<br>
Any ideas?<br>
<span class="HOEnZb"><font color="#888888"><br>
--renato<br>
</font></span></blockquote></div><br></div>