[PATCH] D10140: ARMLoadStoreOptimizer: Rewrite LDM/STM matching logic.

Renato Golin renato.golin at linaro.org
Fri Jul 3 04:54:58 PDT 2015


Hi Matthias,

I'll wait for James' reviews, as this patch is too big and dense and he already had a look at it. In the future, please split the cosmetics (like renaming dl to DL) from feature patches. Each one of the bullets in your description would have been, separately, an easy patch to review.

If James is happy with the patch as is, I won't ask you to split it. But please do next time.

cheers,
--renato


Repository:
  rL LLVM

http://reviews.llvm.org/D10140







More information about the llvm-commits mailing list