[PATCH] [ARM] ARMLoadStoreOpt::UpdateBaseRegUses should stop on def
john.brawn at arm.com
Fri Jun 19 07:10:37 PDT 2015
Hi MatzeB, rengolin,
When UpdateBaseRegUses sees an instruction that defines the base register it must stop, as the base register value it is updating is no longer live. Ideally we would already have seen the register be killed (which is already checked for), but the kill flags may be inaccurate and we have to account for this.
@@ -444,7 +444,7 @@
- if (MBBI->killsRegister(Base))
+ if (MBBI->killsRegister(Base) || MBBI->definesRegister(Base))
// Register got killed. Stop updating.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 419 bytes
Desc: not available
More information about the llvm-commits