Page MenuHomePhabricator

[ThinLTO] Import globals recursively
Needs ReviewPublic

Authored by evgeny777 on Jan 30 2020, 5:40 AM.

Details

Diff Detail

Event Timeline

evgeny777 created this revision.Jan 30 2020, 5:40 AM

FYI we had to disable a related variable importing patch due to some widespread order of magnitude increases in thin link time (see D73724). Until that is addressed, this should go under an option off by default, or just held off.

FYI we had to disable a related variable importing patch due to some widespread order of magnitude increases in thin link time (see D73724). Until that is addressed, this should go under an option off by default, or just held off.

I see. Is it possible to reproduce this on some publicly available code base? Like chromium for example

FYI we had to disable a related variable importing patch due to some widespread order of magnitude increases in thin link time (see D73724). Until that is addressed, this should go under an option off by default, or just held off.

I see. Is it possible to reproduce this on some publicly available code base? Like chromium for example

Not sure, I don't know offhand how to build that one. Going to see if I can root cause where the huge increases come from exactly, will get back.