[llvm-dev] [LLVMdev] Notice for out-of-tree maintainer: RegisterScheduler::setDefault removed

Hans Wennborg via llvm-dev llvm-dev at lists.llvm.org
Thu Feb 11 18:02:12 PST 2016


(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.


More information about the llvm-dev mailing list