RE: [PATCH, rs6000] Fix PR83789: __builtin_altivec_lvx fails for powerpc for altivec-4.c

To: Peter Bergner <berg...@vnet.ibm.com> Cc: GCC Patches <gcc-patches@gcc.gnu.org>; Kaushik Phatak <kaushik.pha...@kpit.com>; Bill Schmidt <wschm...@linux.vnet.ibm.com> Subject: Re: [PATCH, rs6000] Fix PR83789: __builtin_altivec_lvx fails for powerpc for altivec-4.c Hi! On

RE: [PATCH PR83789] Fix for Altivec builtin failure

Hi, Thanks for your quick reply. >> I think you should use altivec_lvx_v4si_2op_si instead? I will look into this. I had used v4si_internal as this was generated in older versions (> And the same needs to be done for v8hi, v16qi, v4sf, maybe more? I did observe some other testcase

[PATCH PR83789] Fix for Altivec builtin failure

. Regards, Kaushik M Phatak 2018-02-01 Kaushik Phatak <kaushik.pha...@kpit.com> PR target/83789 * config/rs6000/rs6000.c (altivec_expand_builtin): Provide support for 32-bit target for altivec builtin. Index: gcc/config/rs6000/rs

RE: [PATCH : RL78] Disable interrupts during hardware multiplication routines

ecific information about these registers from the G13 variant of the RL78 target. We can try and request Renesas to add information about the same along with the option required for this. Nick, do you have any thoughts on this? (assuming this version of patch is closer to acceptance) The pa

[PATCH: RL78] Optimize libgcc routines using clrw and clrb

, Kaushik p.s. Kindly ignore any disclaimers at end of this e-mail as they are auto-inserted. Apologies for the same. 2016-04-06 Kaushik Phatak <kaushik.pha...@kpit.com> * config/rl78/bit-count.S: Use clrw/clrb where possible. * config/rl78/cmpsi2.S: Likewise. * confi

RE: [PING PATCH: RL78] Optimize libgcc routines using clrw and clrb

Hi Nick, Please let me know if you have had a chance to look at the following patch, https://gcc.gnu.org/ml/gcc-patches/2016-02/msg00415.html Best Regards, Kaushik -Original Message- From: Kaushik Phatak Sent: Friday, February 05, 2016 6:26 PM To: 'gcc-patches@gcc.gnu.org' <

[PATCH: RL78] Optimize libgcc routines using clrw and clrb

, Kaushik p.s. Kindly ignore any disclaimers at end of this e-mail as they are auto-inserted. Apologies for the same. 2016-02-05 Kaushik Phatak <kaushik.pha...@kpit.com> * config/rl78/bit-count.S: Use clrw/clrb where possible. * config/rl78/cmpsi2.S: Li

RE: [PATCH : RL78] Disable interrupts during hardware multiplication routines

correctly in hex along with their register name references (, etc.) Please let me know if this updated patch is OK. Best Regards, Kaushik gcc/ChangeLog 2015-12-07 Kaushik Phatak <kaushik.pha...@kpit.com> * config/rl78/rl78.c (rl78_expand_prologue): Save the MDUC r

RE: [PATCH : RL78] Disable interrupts during hardware multiplication routines

for -mg13 -msim -msave-mduc-in-interrupts Best Regards, Kaushik gcc/ChangeLog 2015-08-27 Kaushik Phatak kaushik.pha...@kpit.com * config/rl78/rl78-real.md (movqi_from_mduc,movhi_from_mdal, movhi_from_mdah,movhi_from_mdbl,movhi_from_mdbh,movhi_from_mdcl, movhi_from_mdch

[PATCH: RL78] libgcc fixes for divmodsi, divmodhi and divmodqi

, however the one in divmodsi is critical as the processor runs away to undefined space and crashes. This is regression tested for RL78 -msim. Please let me know if it is OK to commit. Best Regards, Kaushik Changelog: 2015-08-21 Kaushik Phatak kaushik.pha...@kpit.com * config/rl78

[PATCH : RL78] Disable interrupts during hardware multiplication routines

in main code as well as interrupts. This patch has been regression tested with simulator as well as hardware. Please review the same and let me know if OK to commit? Best Regards, Kaushik Phatak 2015-06-05 Kaushik Phatak kaushik.pha...@kpit.com * config/rl78/rl78.md (mulhi3_g13): Disable

RE: [Patch: RL78] Add support for 64-bit doubles

Hi DJ, I assume this should be doubles not double though... I had made this change as suggested, however we had some issue while integrating the tools with our eclipse plug-ins. The RX toolchain has a similar patch where the option is doubles while the folder name is double. Would it be ok to

RE: [Patch: RL78] Add support for 64-bit doubles

Hi DJ, This is OK. Thanks! Do you need someone to commit it for you? Thanks, I can commit these changes. +MULTILIB_DIRNAMES = g10 64-bit-double I assume this should be doubles not double though... Yes, will make that change and commit it. Best Regards, Kaushik

RE: [Patch: RL78] Add support for 64-bit doubles

/2014/msg00151.html https://sourceware.org/ml/binutils/2014-05/msg00140.html Let me know if below patch is ok to commit (also attached). Thanks, Kaushik 2014-05-26 Kaushik Phatak kaushik.pha...@kpit.com * config/rl78/rl78.h (TARGET_CPU_CPP_BUILTINS): Define

RE: [Patch: RL78] Add support for 64-bit doubles

to commit. Thanks, Kaushik 2014-05-14 Kaushik Phatak kaushik.pha...@kpit.com * config/rl78/rl78.h (TARGET_CPU_CPP_BUILTINS): Define __RL78_64BIT_DOUBLES__ or __RL78_32BIT_DOUBLES__. (ASM_SPEC): Pass -m64bit-doubles or -m32bit-doubles on to the assembler

RE: [Patch: RL78] Add support for 64-bit doubles

. Regards, Kaushik 2014-05-12 Kaushik Phatak kaushik.pha...@kpit.com * config/rl78/rl78.h (TARGET_CPU_CPP_BUILTINS): Define __RL78_64BIT_DOUBLES__ or __RL78_32BIT_DOUBLES__. (ASM_SPEC): Pass -m64bit-doubles or -m32bit-doubles on to the assembler

[Patch: RL78] Add support for 64-bit doubles

. Thanks Best Regards, Kaushik 2013-03-06 Kaushik Phatak kaushik.pha...@kpit.com * config/rl78/rl78.h (TARGET_CPU_CPP_BUILTINS): Define __RL78_64BIT_DOUBLES__ or __RL78_32BIT_DOUBLES__. (ASM_SPEC): Pass -m64bit-doubles or -m32bit-doubles on to the assembler

RE: rl78-elf: Compilation broken due to missing constraint

the changelog mentions the addition of the new U constraint, but the actual patch is missing it. Yes, I think you got it right. I have resubmitted this part of the patch again and confirmed the commit in svn. Best Regards, Kaushik

RE: rl78-elf: Compilation broken due to missing constraint

I guess you forgot a small patch to constraints.md, because since this commit which uses a new U constraint, the rl78-elf target won't build: 2013-05-31 Kaushik Phatak kaushik.pha...@kpitcummins.com * config/rl78/rl78.md (mulqi3,mulhi3): New define_expands. (*mulqi3_rl78

RE: new mul* patterns U constraint in rl78

Hi DJ, umul is used when the insn takes two operands in A and X, and computes a value into AX. macax is used when the insn pattern itself reads from two virtual registers and writes to a virtual register (i.e. V = V op V), but clobbers AX anx BC in the process. Thanks for the

RE: new mul* patterns U constraint in rl78

+;; [(set_attr valloc umul)] I think this one needs to be macax also, since the constraints have v in them but the opcodes use ax. Other than that, it's OK. Committed with above change. Thanks for the quick review. Regards, Kaushik

RE: new mul* patterns U constraint in rl78

that be the correct setting as 'macax' is used for the other SI multiplication insns which seem to also include accumulation? Please let me know if OK. Thanks Regards, Kaushik 2013-06-10 Kaushik Phatak kaushik.pha...@kpitcummins.com * config/rl78/constraints.md (U): New constraint

RE: [PATCH:RL78] Add new insn for mulqi3 and mulhi3

Ping. Can I commit this with below changes? Thanks, Kaushik -Original Message- From: Kaushik Phatak Sent: 20 May 2013 20:17 To: gcc-patches@gcc.gnu.org Cc: 'Richard Henderson'; DJ Delorie (d...@redhat.com) Subject: RE: [PATCH:RL78] Add new insn for mulqi3 and mulhi3 Hi Richard, Thanks

RE: [PATCH:RL78] Add new insn for mulqi3 and mulhi3

of this patch. Let me know if OK to commit the same. Regards, Kaushik 2013-05-20 Kaushik Phatak kaushik.pha...@kpitcummins.com * config/rl78/rl78.md (mulqi3,mulhi3): New define_expands. (mulqi3_rl78,mulhi3_rl78,mulhi3_g13): New define_insns. Index: gcc/config/rl78/rl78

[PATCH:RL78] Add new insn for mulqi3 and mulhi3

with no additional regressions. Kindly review the same. Thanks, Kaushik 2013-05-15 Kaushik Phatak kaushik.pha...@kpitcummins.com * config/rl78/rl78.md (mulqi3,mulhi3): New define_expands. (mulqi3_rl78,mulhi3_rl78,mulhi3_g13): New define_insns. Index: gcc/config/rl78/rl78.md

RE: [Patch:RL78] Fix hardware multiply on G13 target

, Kaushik 2013-05-13 Kaushik Phatak kaushik.pha...@kpitcummins.com * config/rl78/rl78.md (mulsi3_g13): Add additional 'nop' required in multiply-accumulate mode --- /home/fsfsrc/gcc-4.8.0-20121219/gcc/config/rl78/rl78.md 2013-01-25 16:26:27.0 +0530 +++ /home

[Patch:RL78] Fix hardware multiply on G13 target

know if this is OK to commit. Regards, Kaushik 2013-05-10 Kaushik Phatak kaushik.pha...@kpitcummins.com * config/rl78/rl78.md (mulsi3_g13): Add additional 'nop' required in multiply-accumulate mode diff -uprN /home/kpit/fsfsrc/gcc-4.8.0-20121219/gcc/config/rl78

RE: [RFA] New port: CR16: BFD Changes required by the gdb port

2013-01-15 Kaushik Phatak kaushik.pha...@kpitcummins.com * configure.ac (cr16-*-*): Adding cr16 target. * configure (cr16-*-*): Regenerate. Index: configure === RCS file: /cvs/src/src/configure,v retrieving revision

RE: [RFA 1/5] New port: CR16: Remove gdb from noconfigdirs in configure.ac

Kaushik Phatak kaushik.pha...@kpitcummins.com Changelog * configure.ac (noconfigdirs): Remove gdb. * configure: Regenerate. --- ./gdb_src.orig/configure2012-06-28 17:20:52.0 +0530 +++ ./gdb_src/configure 2012-08-17 16:56:10.0 +0530 @@ -3447,7 +3447,7

[RFA 1/5] New port: CR16: Remove gdb from noconfigdirs in configure.ac

Hi, This patch is one of patch set to add a new port (National Instruments CR16) in gdb. This patch will, - Remove gdb from noconfigdirs in top-level configure.ac. - Add target-lobgloss - Make target OS independent in config.sub OK for gcc and binutils? Regards, Kaushik 2012-10-04 Kaushik

RE: [Patch : H8300] Bug fix for bit insn and minor tweaks to insns

Hi Jeff, Thanks for the quick review. the right test is rtx_equal_p(operands[0], operands[1]) Committed with above changes to the bsetqi_msx, bclrqi_msx and bnotqi_msx patterns. Thanks Regards, Kaushik Phatak www.kpitgnutools.com -Original Message- From: Jeff Law [mailto:l

[Patch : H8300] Bug fix for bit insn and minor tweaks to insns

which occurs at '-O1'. The other insn's are reordered to give preference to bit instructions using existing constraints. Ok to apply? Thanks Regards, Kaushik Phatak www.kpitgnutools.com 2011-06-09 Kaushik Phatak kaushik.pha...@kpitcummins.com * config/h8300/h8300.md (bsetqi_msx