{"push_id":556500813,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"f9cc07456f8bd165eba1c934d32e98be9a4088c5","before":"75a00557a05f30d21a9e29ab28004058a7117315","commits":[{"sha":"f9cc07456f8bd165eba1c934d32e98be9a4088c5","author":{"email":"1a6d70c497b881333962fb40612e3f3827d7c1cd@philipreames.com","name":"Philip Reames"},"message":"Revert GCStrategy ownership changes\n\nThis change reverts the interesting parts of 226311 (and 227046).  This change introduced two problems, and I've been convinced that an alternate approach is preferrable anyways.\n\nThe bugs were:\n- Registery appears to require all users be within the same linkage unit.  After this change, asking for \"statepoint-example\" in Transform/ would sometimes get you nullptr, whereas asking the same question in CodeGen would return the right GCStrategy.  The correct long term fix is to get rid of the utter hack which is Registry, but I don't have time for that right now.  227046 appears to have been an attempt to fix this, but I don't believe it does so completely.\n- GCMetadataPrinter::finishAssembly was being called more than once per GCStrategy.  Each Strategy was being added to the GCModuleInfo multiple times.\n\nOnce I get time again, I'm going to split GCModuleInfo into the gc.root specific part and a GCStrategy owning Analysis pass.  I'm probably also going to kill off the Registry.  Once that's done, I'll move the new GCStrategyAnalysis and all built in GCStrategies into Analysis.  (As original suggested by Chandler.)  This will accomplish my original goal of being able to access GCStrategy from Transform/  without adding all of the builtin GCs to IR/.","distinct":true,"url":"https://api.github.com/repos/llvm-project/llvm-project/commits/f9cc07456f8bd165eba1c934d32e98be9a4088c5"}]}