On 3/5/18 8:55 AM, Segher Boessenkool wrote: > On Sat, Mar 03, 2018 at 10:55:28PM -0600, Peter Bergner wrote: >> The simple fix here is to just verify the memory_operand is not an altivec >> mem operand before calling rs6000_emit_le_vsx_move. >> >> This passed bootstrap and regtesting on powerpc64le-linux with no >> regressions. Ok for trunk? > > Yes this looks correct, okay for trunk. Thanks. But it is very > non-obvious; maybe a comment will help, or the code can be restructured > a bit?
As we discussed offline, I committed the patch with an added comment. Thanks. Peter