RE: [PATCH 5/7] [ARC] Enable indexed loads for elf targers.

2017-07-17 Thread Claudiu Zissulescu
> The change looks fine, but it would be nice if the commit message > explained _why_ we are default off for Linux and on for Elf, I think > more text in the commit message on this sort of thing will help future > developers understand why things are the way they are. > Committed with suggested a

RE: [PATCH 5/7] [ARC] Enable indexed loads for elf targers.

2017-07-13 Thread Claudiu Zissulescu
> The change looks fine, but it would be nice if the commit message > explained _why_ we are default off for Linux and on for Elf, I think > more text in the commit message on this sort of thing will help future > developers understand why things are the way they are. > This explanation is quite

Re: [PATCH 5/7] [ARC] Enable indexed loads for elf targers.

2017-07-13 Thread Andrew Burgess
* Claudiu Zissulescu [2017-06-01 15:34:55 +0200]: > gcc/ > 2017-02-28 Claudiu Zissulescu > > * config/arc/arc.opt (mindexed-loads): Use initial value > TARGET_INDEXED_LOADS_DEFAULT. > (mauto-modify-reg): Use initial value > TARGET_AUTO_MODIFY_REG_DEFAULT. > * co