Auto merge of #6194 - dwijnand:todo-remove-Compilation.native_dirs, r=dwijnand
Auto merge of #6194 - dwijnand:todo-remove-Compilation.native_dirs, r=dwijnand

Remove the remote TODO on Compilation.native_dirs

It's still used, see tests:

* build_script::doctest_receives_build_link_args
* build_script::rename_with_link_search_path
* run::run_with_library_paths
* run::library_paths_sorted_alphabetically

Originally I was trying to action https://github.com/rust-lang/cargo/pull/792#discussion_r226826431 with this PR

vojtechkral/cargo,tshepang/cargo,eminence/cargo,rust-lang/cargo,tsurai/cargo,rustyhorde/cargo,nipunn1313/cargo,tshepang/cargo,vojtechkral/cargo,nipunn1313/cargo,eminence/cargo,nipunn1313/cargo,tsurai/cargo,tsurai/cargo,vojtechkral/cargo,rust-lang/cargo,tsurai/cargo,rustyhorde/cargo,tshepang/cargo,nipunn1313/cargo,rust-lang/cargo,rustyhorde/cargo,rust-lang/cargo,eminence/cargo,vojtechkral/cargo,rustyhorde/cargo,eminence/cargo,tsurai/cargo,tshepang/cargo