[llvm-commits] [llvm] r157015 - in /llvm/trunk: ./ autoconf/ test/ test/ExecutionEngine/MCJIT/

[Bendersky, Eli]

> Hi Danil,
> 
> Do these really need to be copied into a new directory for the MCJIT? It
> seems we could run them for the MCJIT with a second RUN line in the
> original files instead. I semi-recall some discussion previously about doing
> something like that for some of the ELF tests. Eli, do you remember any of
> the details of that?

Jim,

We examined a number of ways to avoid duplicating the tests, but all of them stumbled on problems (i.e. the second RUN was infeasible because of limitations in lit for expecting failures for some RUNs). Eventually after some discussions with Daniel, I've started working on a more comprehensive change to lit for adding a SUBTEST  feature. An initial patch has already been reviewed by Daniel and I hope to have a fixed patch in the next few days. Once that's in lit, we'll be able to leverage this new functionality to cleanly run ExecutionEngine tests on both JIT and MCJIT with the same test file, without a need for duplication.

Eli




---------------------------------------------------------------------
Intel Israel (74) Limited

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.