On Sun, 19 Aug 2012, Oleg Endo wrote:
> This is what has been done so far on the SH side for 4.8.
> I hope it's OK.

Wow, that is quite impressive (and a nice write-up also)!

I see this was already approved, but allow me to suggest some
minor editorial comments...

Index: htdocs/gcc-4.8/changes.html
===================================================================
+    <li>The default alignment settings have been reduced to be less aggresive.

"aggressive"

+      <li>Minor tweaks for code around software atomic sequences that are
+      enabled by <code>-msoft-atomic</code>.</li>

"code for"?  Not sure...

+      <li>A new option <code>-menable-tas</code> will make the compiler
+      generate the <code>tas.b</code> instruction for the
+      <code>__atomic_test_and_set</code> built-in function.</li>

A naive question: Why is this not on by default?  Or is it under
certain circumstances?

+    <li>The <code>fmac</code> instruction will now be emitted by the
+    <code>fmaf</code> standard and built-in function.</li>

"built-in standard function", perhaps?

+    <li>The <code>-mfused-madd</code> option has been depricated in favor of

"deprecated"

+    <li>Added new options <code>-mfsrra</code> and <code>-mfsca</code> to allow
+    the compiler using the <code>fsrra</code> and <code>fsca</code>
+    instructions on CPUs other than SH4A.</li>

How about adding "...SH4A (where they are already on by default)" or
similar?

Gerald

Reply via email to