Remove gc.root's performCustomLowering
authorPhilip Reames <listmail@philipreames.com>
Wed, 28 Jan 2015 19:28:03 +0000 (19:28 +0000)
committerPhilip Reames <listmail@philipreames.com>
Wed, 28 Jan 2015 19:28:03 +0000 (19:28 +0000)
commitafe3498413029d158aa6b0da6b05abc5c307dea6
treec540f016e509e18cb693095dbec6eb9891ca03f5
parentb268ba8022747423db4930bdeb4a4bdbff5d6066
Remove gc.root's performCustomLowering

This is a refactoring to restructure the single user of performCustomLowering as a specific lowering pass and remove the custom lowering hook entirely.

Before this change, the LowerIntrinsics pass (note to self: rename!) was essentially acting as a pass manager, but without being structured in terms of passes. Instead, it proxied calls to a set of GCStrategies internally. This adds a lot of conceptual complexity (i.e. GCStrategies are stateful!) for very little benefit. Since there's been interest in keeping the ShadowStackGC working, I extracting it's custom lowering pass into a dedicated pass and just added that to the pass order. It will only run for functions which opt-in to that gc.

I wasn't able to find an easy way to preserve the runtime registration of custom lowering functionality. Given that no user of this exists that I'm aware of, I made the choice to just remove that. If someone really cares, we can look at restoring it via dynamic pass registration in the future.

Note that despite the large diff, none of the lowering code actual changes. I added the framing needed to make it a pass and rename the class, but that's it.

Differential Revision: http://reviews.llvm.org/D7218

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@227351 91177308-0d34-0410-b5e6-96231b3b80d8
docs/GarbageCollection.rst
include/llvm/CodeGen/GCStrategy.h
include/llvm/CodeGen/Passes.h
include/llvm/InitializePasses.h
lib/CodeGen/CMakeLists.txt
lib/CodeGen/GCRootLowering.cpp
lib/CodeGen/Passes.cpp
lib/CodeGen/ShadowStackGC.cpp
lib/CodeGen/ShadowStackGCLowering.cpp [new file with mode: 0644]