labath added a comment.
While I'm not opposed to moving "lldbsuite" to some place closer to the tests,
one thing I'd like to get out of this is to have the actual tests stand out
more prominently. For that reason, I wouldn't want to just take the "lldbsuite"
folder and "drop" it into `test/API`. In fact, ideally I'd like to get rid of
the redundant nested `test` subfolder, and put the tests directly into
`test/API`.
However, like I said in the other review, if lua is going to become a thing, we
should evaluate how will this layout work with additional scripting languages:
I see several options here:
- make a subfolder for each language (`test/API/{Python,Lua}`)
- ditch the "API" part and put these at the top level (`test/{Python,Lua}`)
- keep Python in "API", but put lua in Lua (`test/{API,Lua}`) -- besides
backward compat, this would kind of signal that "python" is our preferred
vessel for writing tests
- ???
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71151/new/
https://reviews.llvm.org/D71151
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
