[PATCH] RegAllocGreedy: Allow target to specify register class ordering.

Andrew Trick atrick at apple.com
Wed Mar 25 21:25:03 PDT 2015


Ok. The comment is misleading though. It makes it sounds as though getClassPriority can be used to prioritize "big" live ranges. I think it instead allows register class order to take precedence over live range size.

Also, this really puts the burden on the target to assign priorities that make sense in the context of the regalloc algorithm. I think it's ok to have that back door, but it seems like there should be a use-at-your-own risk disclaimer, no? I would indicate that the default implementation should be used unless the hook is really needed to improve allocation.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D8626

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list