Re: Swap SECTION_EXCLUDE and SECTION_MACH_DEP

2012-03-12 Thread Sriraman Tallam
 Already submitted by gjl in rev. 185259.

-Sri.

On Mon, Mar 12, 2012 at 10:27 AM, Richard Henderson r...@redhat.com wrote:
 On 03/12/12 10:26, Sriraman Tallam wrote:
 Hi Richard,

   Is this ok for trunk? Asking you because you approved the patch the
 first time around.

 Thanks,
 -Sri.

 On Fri, Mar 9, 2012 at 3:10 PM, Sriraman Tallam tmsri...@google.com wrote:
 Please see:
 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52545

 SECTION_EXCLUDE should not take up bits reserved for SECTION_MACH_DEP

 Patch:

        PR 52545
        * output.h (SECTION_MACH_DEP)
        (SECTION_EXCLUDE):  Swap values.


 Yes, this patch is ok


 r~


Re: Swap SECTION_EXCLUDE and SECTION_MACH_DEP

2012-03-09 Thread Sriraman Tallam
Please see:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52545

SECTION_EXCLUDE should not take up bits reserved for SECTION_MACH_DEP

Patch:

PR 52545
* output.h (SECTION_MACH_DEP)
(SECTION_EXCLUDE):  Swap values.



Index: output.h
===
--- output.h(revision 185120)
+++ output.h(working copy)
@@ -446,8 +446,8 @@
 #define SECTION_STYLE_MASK 0x60/* bits used for SECTION_STYLE */
 #define SECTION_COMMON   0x80  /* contains common data */
 #define SECTION_RELRO   0x100  /* data is readonly after relocation
processing */
-#define SECTION_MACH_DEP 0x200 /* subsequent bits reserved for target 
*/
-#define SECTION_EXCLUDE  0x400  /* discarded by the linker */
+#define SECTION_EXCLUDE  0x200  /* discarded by the linker */
+#define SECTION_MACH_DEP 0x400 /* subsequent bits reserved for target 
*/

Ok to submit?

Thanks.
-Sri.


On Fri, Mar 9, 2012 at 3:07 PM, Sriraman Tallam tmsri...@google.com wrote: