Elimiates lots of unused code.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
- Build Status
Buildable 38517 Build 38516: arc lint + arc unit
Event Timeline
Thanks! That's a great job, and certainly saves me some trouble in pushing my patch forward, though I'm kinda surprised we don't need site-packages path there after all.
Please wait for others to review it as well.
Yes that's surprising. Seems like it is better to move the install() call in your change to lldb/CMakeLists.txt. To be together with the call to finishSwigWrapperClasses.py.
Yay, tons of nasty python code going down the drain. This looks fine, but please take it slowly when landing these patches. Waiting a day or two before proceeding with the next patch will make it easier to fix things up if any of them cause problems for some people.
I guess that's because this script puts its output in a temporary location, and the "finish" script then copies it to the site-packages folder. I think this was mainly done because we were doing some post-processing of the swig-generated code, which we don't do any more thankfully, so it may be possible to have this put the code straight into the final location, but it's better to take things one slowly.
lldb/trunk/scripts/CMakeLists.txt | ||
---|---|---|
5 ↗ | (On Diff #221790) | Doesn't this now include the lldb-private headers now? Is that intended? |
lldb/trunk/scripts/CMakeLists.txt | ||
---|---|---|
5 ↗ | (On Diff #221790) | Oh sorry. Will send a change to fix. Is there an easy way to exclude a pattern in file() call? |
lldb/trunk/scripts/CMakeLists.txt | ||
---|---|---|
5 ↗ | (On Diff #221790) | You can use EXCLUDE but I'm not sure that works with file(GLOB). It might be better just to enumerate all the headers you want to use instead of globbing. |
lldb/trunk/scripts/CMakeLists.txt | ||
---|---|---|
5 ↗ | (On Diff #221790) | +1 |