RE: [PATCH 6/7] [ARC] Deprecate mexpand-adddi option.

2017-07-17 Thread Claudiu Zissulescu
> This looks fine, though the commit message tells me it's not a good > idea, but it would be nice to know _why_ it's not good. Might be nice > to know for future reference. Committed with additional comments. Thank you, Claudiu

RE: [PATCH 6/7] [ARC] Deprecate mexpand-adddi option.

2017-07-13 Thread Claudiu Zissulescu
> > This looks fine, though the commit message tells me it's not a good > idea, but it would be nice to know _why_ it's not good. Might be nice > to know for future reference. > Again LRA, expand time subregs are not handled by LRA, as far as I can tell. Moreover, this option is by default off,

Re: [PATCH 6/7] [ARC] Deprecate mexpand-adddi option.

2017-07-13 Thread Andrew Burgess
* Claudiu Zissulescu [2017-06-01 15:34:56 +0200]: > From: claziss > > Emitting subregs in the expand is not a good idea. Deprecate this > option. > > gcc/ > 2017-04-26 Claudiu Zissulescu > > * config/arc/arc.md (adddi3): Remove support for mexpand-adddi > option. > (subd

[PATCH 6/7] [ARC] Deprecate mexpand-adddi option.

2017-06-01 Thread Claudiu Zissulescu
From: claziss Emitting subregs in the expand is not a good idea. Deprecate this option. gcc/ 2017-04-26 Claudiu Zissulescu * config/arc/arc.md (adddi3): Remove support for mexpand-adddi option. (subdi3): Likewise. * config/arc/arc.opt (mexpand-adddi): Deprecat