Hans Wennborg via llvm-dev
2016-Feb-12 02:02 UTC
[llvm-dev] [LLVMdev] Notice for out-of-tree maintainer: RegisterScheduler::setDefault removed
(Now with the contemporary llvm-dev list; sorry for the noise.) On Thu, Feb 11, 2016 at 6:01 PM, Hans Wennborg <hans at chromium.org> wrote:> On Thu, Feb 11, 2016 at 4:37 PM, Mehdi Amini <mehdi.amini at apple.com> wrote: >> >>> On Feb 11, 2016, at 4:24 PM, Hans Wennborg <hans at chromium.org> wrote: >>> >>> On Tue, Jul 28, 2015 at 7:58 AM, Mehdi Amini <mehdi.amini at apple.com> wrote: >>>> Just a notice for out-of-tree maintainer, I removed >>>> RegisterScheduler::setDefault in r243388. It was used to override the >>>> default command line option that selects the DAG scheduler to use. >>>> >>>> Instead of hacking into the command line plumbing, a target that wants more >>>> control over the selection of the scheduler than it already has with >>>> setSchedulingPreference has a SubTargetHook “getDAGScheduler” that can be >>>> overridden. The original motivation for this change was to remove some >>>> mutable global state. >>> >>> Might be worth mentioning in the release notes? >> >> Yes, patch: > > Thanks! r260643.