[llvm] [ElimAvailExtern] Add an option to allow to convert global variables in a specified address space to local (PR #144287)

[Shilei Tian via llvm-commits]

shiltian wrote:

> This is not the default behavior. This pass mostly exists to drop definitions, not to convert to local. The conversion was added for contextual profiling. So the reference to this behavior in the summary is a bit odd.

Thanks for the information. I'll correct the description.

>  Do you _also_ want to convert functions to locals?

Yes but that is controlled by the existing option `avail-extern-to-local`.

> If so, should this be under the same option(s)?

My understanding is `avail-extern-to-local` is for function and the new option is for global variable. Do you think it's a better idea to check both options for global variables?

https://github.com/llvm/llvm-project/pull/144287