Fix for PR14824: Optimization arm_ldst_opt inserts newly generated instruction vldmia at incorrect position

Renato Golin renato.golin at linaro.org
Tue Apr 2 01:28:41 PDT 2013


On 27 March 2013 12:50, Stepan Dyatkovskiy <stpworld at narod.ru> wrote:

> Hello guys,
> Well I introduced unit-regs-in-use set, and reg-to-unit-regs map. Now each
> time we add load operation it just resolves unit-regs affected and checks
> case when new operation overrides some previous one.
> If I detect the "override" fact, well I scan all previous load operations
> and remove ones were overridden.
>

Hi Stepan,

Sorry for the delay, but the patch looks good as far as I can tell in this
early morning review. ;)

If no one else objects, and you ran all appropriate tests (including
check-all and possibly the test-suite on at least one platform), I'm ok
with it.

cheers,
--renato
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130402/37e9942f/attachment.html>


More information about the llvm-commits mailing list