On 9/26/19 12:00 PM, Segher Boessenkool wrote:
Hi Will,

On Thu, Sep 26, 2019 at 10:40:29AM -0500, will schmidt wrote:
Update our (rs6000) vector load built-ins with the PURE attribute.  These
were previously given the MEM attribute, which meant that redundant loads
surrounding the built-in calls could not be eliminated in earlier passes
since they were defined as having the potential to touch memory.
2019-09-26  Will Schmidt <will_schm...@vnet.ibm.com>
        * config/rs6000/rs6000-builtin.def: ( LVSL LVSR LVEBX LVEHX
                                              ^--- stray space

Please put commas between the items, too?

The patch is okay for trunk.  Thank you!

I wonder whether we should also consider a backport to 9, when we started expanding these earlier.  Thoughts?

Bill

Segher

Reply via email to