Hi Prathamesh,

Prathamesh Kulkarni wrote:
The attached patch similarly adds target pragmas to the test as previous patch.
Does it look OK ?

Thanks for revising Matthew's patch.

Yes, LGTM. – However, except for 'Daily bump.', it seems to be uncommon to have a '.' at the end of the first line (summary) of the git commit; hence, consider removing it.

Tobias

Reply via email to