> This looks fine. Thanks for all your effort revising this patch.
>
> Andrew
>
Committed r242425.
Thank you for your review,
Claudiu
Please find attached the revised patch as requested.
Ok to apply?
Claudiu
gcc/
2016-05-09 Claudiu Zissulescu
* config/arc/arc-arch.h: New file.
* config/arc/arc-arches.def: Likewise.
* config/arc/arc-cpus.def: Likewise.
* config/arc/arc-options.def: Likewise.
> HEAD binutils supports '.cpu NPS400' and HEAD GCC also correctly emits
> '.cpu NPS400' (when -mcpu=nps400 is passed). After your change we no
> longer correctly emit '.cpu NPS400'.
>
Sorry for this miss-understanding. Updating the patch to emit .cpu NSPS400.
Thanks,
Claudiu
* Claudiu Zissulescu [2016-11-10 12:02:34
+0100]:
> Hi,
>
> Please find the revised patch which includes the refurbishing of
> mmpy-option option, and a new comment on DEFAULT_arc_fpu_build
> define. As for the last suggestion, my proposal is to have a latter
> patch on the topic of .cpu, synce
Hi,
Please find the revised patch which includes the refurbishing of
mmpy-option option, and a new comment on DEFAULT_arc_fpu_build
define. As for the last suggestion, my proposal is to have a latter
patch on the topic of .cpu, synced with a related binutils patch.
OK to apply?
Claudiu
gcc/
2016
Andrew,
> you create the TargetVariable arc_mpy_option, however, I think it
> would be neater to fold this variable into the mmpy-option as it was
> before, but, changing the type to Enum. This would allow the big
> option checking switch to be removed from arc-common.c, which I think
> is a win
* Claudiu Zissulescu [2016-10-31 16:46:17
+0100]:
> Please find the updated patch.
>
> What is new:
> - The .def files are having a comment block on how to add new lines.
> - The arc_seen_option is not used.
> - The arc_cpu* variables are not used.
>
> Please let me know if I miss something,
Hi Andrew,
Please find the updated patch.
What is new:
- The .def files are having a comment block on how to add new lines.
- The arc_seen_option is not used.
- The arc_cpu* variables are not used.
Please let me know if I miss something,
Claudiu
gcc/
2016-05-09 Claudiu Zissulescu
*
Hi Andrew,
> Sorry it's taken s long to review this patch.
>
This is understandable, it is a quite large patch and changes a lot of items.
> In general I like this, and think it's a step in the right direction.
> I have a few pretty minor concerns, but hopefully nothing too
> contentious.
>
* Claudiu Zissulescu [2016-10-10 15:26:47
+0200]:
> Hi Andrew,
>
> This is updated patch of the original sent to mailing list some while ago.
>
> What is new:
> - Do not use MULTILIB_REUSE as its semantic changed, and the old one was
> causing issues while building.
> - Update invok
Hi Andrew,
This is updated patch of the original sent to mailing list some while ago.
What is new:
- Do not use MULTILIB_REUSE as its semantic changed, and the old one was
causing issues while building.
- Update invoke.texi documentation adding nps400 option to mcpu.
This patch is imp
11 matches
Mail list logo
