[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