This short-circuits the GetObjCVersion function to avoid iterating through target modules on
non-apple targets. This function is called on every Process::IsDynamicValue call, so this
overhead is not negligible.
Details
Details
- Reviewers
clayborg jingham - Commits
- rG31bcd2bddde3: [AppleObjCRuntime] Don't bother looking for the runtime on non-apple targets
rLLDB251004: [AppleObjCRuntime] Don't bother looking for the runtime on non-apple targets
rL251004: [AppleObjCRuntime] Don't bother looking for the runtime on non-apple targets
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
This is really the fault of AppleObjCRuntimeV1::CreateInstance() and AppleObjCRuntimeV2::CreateInstance() not checking the triple to begin with, but this is a simple solution that will effectively fix both places. Looks good.