On 08/03/2015 04:02 AM, Zoltan Gilian wrote: > > +static inline bool IsEOP(MachineBasicBlock::iterator I) > +{ > + return std::next(I)->getOpcode() == AMDGPU::RETURN; > +} > + Brace should be on previous line and name should be isEOP(). Other than that LGTM