> 1. Please rename SelectAddrMode7. There is no such addressing mode. We used 
> 1, 2, ..., 6 because that's what earlier ARM doc used, but we really should 
> rename them.

Ok

> 2. Another solution for the register pair issue is to assign them fixed 
> physical registers at isel time. The good thing about it is it eliminate the 
> complexity in ARMLoadStoreOpt.

I would be fine with this too. The only problem is that I would have
to manually select the strexd instead of using a simple pattern for
it, but it's always a tradeoff anyway. Let me know which approach is
preferred then.


-- 
Bruno Cardoso Lopes
http://www.brunocardoso.cc
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to