Ping.
On Mon, May 19, 2014 at 11:25 AM, Sriraman Tallam <tmsri...@google.com> wrote: > Ping. > > On Thu, Apr 17, 2014 at 10:41 AM, Sriraman Tallam <tmsri...@google.com> wrote: >> Ping. >> >> On Wed, Feb 5, 2014 at 4:31 PM, Sriraman Tallam <tmsri...@google.com> wrote: >>> Hi, >>> >>> I would like this patch reviewed and considered for commit when >>> Stage 1 is active again. >>> >>> Patch Description: >>> >>> A C++ thunk's section name is set to be the same as the original function's >>> section name for which the thunk was created in order to place the two >>> together. This is done in cp/method.c in function use_thunk. >>> However, with function reordering turned on, the original function's section >>> name can change to something like ".text.hot.<orginal>" or >>> ".text.unlikely.<original>" in function default_function_section in varasm.c >>> based on the node count of that function. The thunk function's section name >>> is not updated to be the same as the original here and also is not always >>> correct to do it as the original function can be hotter than the thunk. >>> >>> I have created a patch to not name the thunk function's section to be the >>> same >>> as the original function when function reordering is enabled. >>> >>> Thanks >>> Sri
