[compiler-rt] r204299 - PGO: Split out initialization of section bounds
Duncan P. N. Exon Smith
dexonsmith at apple.com
Thu Mar 20 11:50:16 PDT 2014
On Mar 19, 2014, at 9:22 PM, Duncan P. N. Exon Smith <dexonsmith at apple.com> wrote:
>
> On 2014 Mar 19, at 21:16, Justin Bogner <mail at justinbogner.com> wrote:
>
>> "Duncan P. N. Exon Smith" <dexonsmith at apple.com> writes:
>>> PGO: Split out initialization of section bounds
>>>
>>> Currently we register instrumentation data at runtime to determine the
>>> bounds of the sections where the data lives. Soon we'll implement
>>> platform-specific linker magic to determine this at link time.
>>>
>>> Move this logic to a separate file, so that our build system can choose
>>> the correct platform-specific code.
>>>
>>> No functionality change intended.
>> ...
>>> Added: compiler-rt/trunk/lib/profile/InstrProfilingDefault.c
>>
>> It's pretty bikeshed-y, but something like "manual" or "register" would
>> be more descriptive than default, especially since this won't be the
>> default on platforms where we can do better.
>
> The other suffixes will be things like Darwin and Elf, so Manual and
> Register don’t line up. Maybe DefaultPlatform? UnknownPlatform? I
> still like Default better than the other options, but I’m open to more
> suggestions :).
Moved them in r204373 to PlatformDarwin and PlatformOther. When the Elf
stuff gets written it can go into PlatformElf.
More information about the llvm-commits
mailing list