After inlining the local static void to which it used to delegate most of the work, the main loop can access *this and we have no need for yet another copy of it or the variables to iterate it. We can also access the result array directly, rather than via a (poorly-named) pointer. It turns out that wrapping QBA in QBAV each time we call a lambda on it costs enough to cause a 50% slowdown, so change the lambda to now take the const refs we have. (Constructing the views only once is as good, for reference.) Change-Id: I81c5996d91415d8a933de714177a89462555ff03 Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>