On Tue, Jul 21, 2020 at 06:37:29PM -0400, David Edelsohn wrote: > On Tue, Jul 21, 2020 at 5:54 PM Segher Boessenkool > <seg...@kernel.crashing.org> wrote: > > always (the target cannot run the resulting code, but we have many other > > options like that, starting with -mcpu=). David, what is your > > preference? > > > > The rs6000 parts of this patch are fine for trunk. Thanks! > > There already is a { target be } selector. This set of testcases > shouldn't be limited to powerpc-linux. Ideally the scan-assembler > strings should have "be" and "le" versions to confirm the correct > results for either.
There are some testcases that are really just for BE (or LE); but yeah, we can just not scan the assembler on the oppositely endianed targets. Very few tests are run tests that only work on one endianness, but that is easy to adjust to as well (wrap some #ifdef around it), and infrequent enough. Okay, thanks, I worry too much :-) Segher