On Thu, Sep 26, 2019 at 12:06:03PM -0500, Bill Schmidt wrote: > On 9/26/19 12:00 PM, Segher Boessenkool wrote: > >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?
Good idea, yeah. After waiting for a week or so to see if problems turn up on trunk, as usual. Segher
