]> granicus.if.org Git - llvm/commit
Adjust how we avoid poll insertion inside the poll function (NFC)
authorPhilip Reames <listmail@philipreames.com>
Tue, 10 Feb 2015 00:04:53 +0000 (00:04 +0000)
committerPhilip Reames <listmail@philipreames.com>
Tue, 10 Feb 2015 00:04:53 +0000 (00:04 +0000)
commit2fc495a300d19b5f294b32e7dde864a945bf2dbc
treeb4bf398acd89bd1b31d7a704ca60a5860c91cbca
parentc0cd86712c4c637b3662c1182b5e07766fc156c0
Adjust how we avoid poll insertion inside the poll function (NFC)

I realized that my early fix for this was overly complicated.  Rather than scatter checks around in a bunch of places, just exit early when we visit the poll function itself.

Thinking about it a bit, the whole inlining mechanism used with gc.safepoint_poll could probably be cleaned up a bit.  Originally, poll insertion was fused with gc relocation rewriting.  It might be worth going back to see if we can simplify the chain of events now that these two are seperated.  As one thought, maybe it makes sense to rewrite calls inside the helper function before inlining it to the many callers.  This would require us to visit the poll function before any other functions though..

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@228634 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Transforms/Scalar/PlaceSafepoints.cpp