On 04/20/2012 03:37 AM, Naveen H. S wrote:
Hi,

Please find attached the patch "h8300_indent.patch" that adapts H8300
machine descriptions to the braced string notation. The changes makes
the "Machine Descriptor" file more easier to read.

It is similar to the following changes in SH target.
http://gcc.gnu.org/ml/gcc-patches/2012-03/msg01315.html

Tested with C testsuite for h8300-elf. No new regressions.
        
ChangeLog

        * config/h8300/h8300.md: Use braced string notation where applicable.
This is fine. Do you have checkin privs for the GCC repository? If not, please request them so you can check in your patches.


Like Mike, I think the braces ought to be indented, but there's a multitude of mis-indented braces in h8300.md already. If you wanted to fix up that aspect of formatting the md file, that'd be a fine follow-up patch.

Jeff

Reply via email to