This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

[PATCH][RX] Patch to correct the functionality of compiler option -falign-labels=n

From: Sandeep Kumar Singh <Sandeep dot Singh2 at kpit dot com>
To: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Cc: "nick clifton (nickc at redhat dot com)" <nickc at redhat dot com>, "dj at delorie dot com" <dj at delorie dot com>
Date: Wed, 11 Jun 2014 12:43:41 +0000
Subject: [PATCH][RX] Patch to correct the functionality of compiler option -falign-labels=n
Authentication-results: sourceware.org; auth=none
Authentication-results: spf=none (sender IP is ) smtp dot mailfrom=Sandeep dot Singh2 at kpit dot com;

Hi,

The RX compiler align wrong labels with option falign-labels=n, where n is number 
of bytes. The relationship between -falign-labels and expected .balign output 
supposed to be:
-falign-labels=4 --> .balign 4
-falign-labels=8 --> .balign 8
-falign-labels=16 --> .balign 16

However, listed pseudo code was the like (without any optimization):
-falign-labels=1 --> .balign 2
-falign-labels=4 --> .balign 16
-falign-labels=8 --> .balign 256
-falign-labels=16 --> .balign 65536

No regression found with this patch.

Please review the patch and let me know if there should be any modifications
in it?

Regards,
Sandeep Kumar Singh,
KPIT Cummins InfoSystems Ltd.
Pune, India

gcc/ChangeLog
2014-06-11  Sandeep Kumar Singh  <Sandeep.Singh2@kpitcummins.com>

        * config/rx/rx.h (Corrected macro LABEL_ALIGN).

Attachment: rx_align_labels.patch
Description: rx_align_labels.patch

Follow-Ups:
- Re: [PATCH][RX] Patch to correct the functionality of compiler option -falign-labels=n
  - From: DJ Delorie

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]