https://bugs.llvm.org/show_bug.cgi?id=39524
Bug ID: 39524
Summary: Replace lli with %lli in RUN lines
Product: Test Suite
Version: trunk
Hardware: PC
OS: Linux
Status: NEW
Severity: enhancement
Priority: P
Component: lit
Assignee: unassignedb...@nondot.org
Reporter: eus...@gmail.com
CC: dan...@zuster.org, llvm-bugs@lists.llvm.org
Created attachment 21063
--> https://bugs.llvm.org/attachment.cgi?id=21063&action=edit
WIP patch for converting all tests to %lli
`lit.cfg.py` defines substitutions for both `lli` and `%lli`, and this is
confusing, especially since there is no actual difference between these two
except that on Windows `%lli` includes an `-mtriple=' flag force-setting ELF
object format.
There is even a FIXME in `test/lit.cfg.py' about this very thing:
> # FIXME: Why do we have both `lli` and `%lli` that do slightly different
> things?
It seems like the majority of tests could be converted to use one or the other.
If all tests can be converted to one substitution, the other one should be
removed.
If there is a minority of tests that depend on the other substitution,
that substitution should be renamed to something special a la `%llc_dwarf` to
clearly indicate that it is a special case and not the default.
I included a patch that converts everything to `%lli`, but I don't know if it
really works because I don't have a Windows machine to test it on.
--
You are receiving this mail because:
You are on the CC list for the bug._______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs
