[PATCH] Remove gc.roots performCustomLowering

[Philip Reames]

Responding to comments.  (Those not specifically responded to will be fixed before submission.)


================
Comment at: docs/GarbageCollection.rst:751
@@ -750,3 @@
-
-* ``llvm.gcwrite``: Substitute a ``store`` instruction.
-
----------------
mjacob wrote:
> I think these three points can stay in because they're still valid.
They're also covered elsewhere in the same file already.

================
Comment at: lib/CodeGen/ShadowStackGCLowering.cpp:234
@@ +233,3 @@
+  //        (which uses a FunctionPassManager (which segfaults (not asserts) if
+  //        provided a ModulePass))).
+  Constant *GV = new GlobalVariable(*F.getParent(), FrameMap->getType(), true,
----------------
mjacob wrote:
> I think it's now possible to add ModulePasses to the llc pass pipeline, so changing this pass to be a ModulePass should be possible.
I think this comment is just stale.  Given I didn't touch any of this code, I'll leave it as is for now.

http://reviews.llvm.org/D7218

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/