On Mon, 2019-01-21 at 18:00 +0000, Tamar Christina wrote: > > > That would need to be aarch64*-*-* to include big-endian. Fixing that here > > and in the other tests is OK under the obvious rule. > > Ah, true, I didn't look at the testcase. I tested the ILP32 case and > committed the fix for both. > > Thanks, > Tamar > > > > > Adding && lp64 (as per Steve's patch below) is OK too if it works. > > > > Thanks, > > Richard
Thanks for fixing this test Tamar. I am going to change the others to use 'aarch64*-*-*' instead of 'aarch64-*-*' since that seems to be the standard method (though I see a few other tests that are not using aarch64 instead of aarch64*). I guess that while you are testing big-endian the target triplet you are using is still aarch64-*-* and not aarch64be-*-*? Otherwise I would have expected you to have more failures. Steve Ellcey sell...@marvell.com