{"push_id":558862730,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"afe3498413029d158aa6b0da6b05abc5c307dea6","before":"b268ba8022747423db4930bdeb4a4bdbff5d6066","commits":[{"sha":"afe3498413029d158aa6b0da6b05abc5c307dea6","author":{"email":"1a6d70c497b881333962fb40612e3f3827d7c1cd@philipreames.com","name":"Philip Reames"},"message":"Remove gc.root's performCustomLowering\n\nThis is a refactoring to restructure the single user of performCustomLowering as a specific lowering pass and remove the custom lowering hook entirely.\n\nBefore 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.\n\nI 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.\n\nNote 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.\n\nDifferential Revision: http://reviews.llvm.org/D7218\n\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@227351 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/llvm-project/llvm/commits/afe3498413029d158aa6b0da6b05abc5c307dea6"}]}