{"comment":{"url":"https://api.github.com/repos/halide/Halide/comments/16956323","html_url":"https://github.com/halide/Halide/commit/0c46a6e8a3973eda3efeefcdf2450c86b7465d79#commitcomment-16956323","id":16956323,"user":{"login":"dsharletg","id":7494796,"avatar_url":"https://avatars.githubusercontent.com/u/7494796?v=3","gravatar_id":"","url":"https://api.github.com/users/dsharletg","html_url":"https://github.com/dsharletg","followers_url":"https://api.github.com/users/dsharletg/followers","following_url":"https://api.github.com/users/dsharletg/following{/other_user}","gists_url":"https://api.github.com/users/dsharletg/gists{/gist_id}","starred_url":"https://api.github.com/users/dsharletg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dsharletg/subscriptions","organizations_url":"https://api.github.com/users/dsharletg/orgs","repos_url":"https://api.github.com/users/dsharletg/repos","events_url":"https://api.github.com/users/dsharletg/events{/privacy}","received_events_url":"https://api.github.com/users/dsharletg/received_events","type":"User","site_admin":false},"position":34,"line":54,"path":"test/hexagon/apps/sobel/sobel.cpp","commit_id":"0c46a6e8a3973eda3efeefcdf2450c86b7465d79","created_at":"2016-04-04T17:06:02Z","updated_at":"2016-04-04T17:06:02Z","body":"Why did this need to be saturating arithmetic? The original input is uint8, it then gets widened to uint16. That should be plenty of headroom for all of the arithmetic to work without saturation, it only needs an extra 2 bits above the input."}}