[PATCH] D25482: [PPC] Allow two post RA schedulers to be in the pipeline and select one depending on the Machine Function's subtarget

Ehsan Amiri via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 1 18:10:49 PDT 2016


amehsan added a comment.

> 
> 
>> 2- targetSchedulesPostRAScheduling
> 
> This seems redundant to me as soon as 1) is implemented.

Maybe I am missing something, but this seems different from (1). Note that addPostRASched is called from addMachinePasses which is not typically overriden by targets. SystemZ has a usecase for targetSchedulesPostRAScheduling which cannot be covered by (1).


https://reviews.llvm.org/D25482





More information about the llvm-commits mailing list