[llvm] Make functions always renamable (PR #115226)

[Teresa Johnson via llvm-commits]

================
@@ -175,6 +175,9 @@ static CalleeInfo::HotnessType getHotness(uint64_t ProfileCount,
 }
 
 static bool isNonRenamableLocal(const GlobalValue &GV) {
+  if (isa<Function>(&GV)) {
----------------
teresajohnson wrote:

Sounds good on continuing discussion on RFC. Thanks in advance for digging into this.

> That said, if we go down the route of the patch proposed here, we need to rename the function by remove that Local.

I'm not sure why. It is still meant to return true if this is a non-renamable local symbol. We just want to fine tune the conditions.

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