[PATCH] D30124: [ARM] Fix insert point for store rescheduling.

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 21 13:10:43 PST 2017


MatzeB accepted this revision.
MatzeB added a comment.
This revision is now accepted and ready to land.

The change itself looks straightforward.
Having the magic number `8` tunable would indeed be nice, unfortunately target features are only boolean (enable or disabled) today; While I would applaud everyone improving the targetfeature system to support arbitrary integer values, I don't think we need to block this bugfix for it esp. since the magic number already existed in the code.


Repository:
  rL LLVM

https://reviews.llvm.org/D30124





More information about the llvm-commits mailing list