llvm/include/llvm/CodeGen/MachineScheduler.h:810: pointless copy of a list ?

[dcb314]

Static analyser cppcheck says:

llvm/include/llvm/CodeGen/MachineScheduler.h:810:51: performance: Function parameter 'Intervals' should be passed by const reference. [passedByValue]

Source code is

explicit ResourceSegments(std::list Intervals)

[JOE1994]

The mentioned constructor isn't used anywhere in the codebase, so I think we can just remove the constructor.
It can be added back in when it is actually needed.