On 02/27/2018 01:51 PM, Håkon Sandsmark wrote:
Thanks for the feedback. I chose to take the example from the bug
report verbatim as the test case.

However, I agree it makes sense to have the simplest possible test
case that reproduces the issue. Here is an updated patch.

Thanks!

+      /* If there is any qualification still in effect, clear it
+       * now; we will be starting fresh with the next capture.  */

For future reference, we don't add * at the beginning of subsequent lines in a comment. I'll correct that in this patch and check it in.

It looks like you don't have a copyright assignment on file with the FSF yet. This patch is small enough not to need one, but if (as I hope) you're thinking to continue contributing to GCC, you might want to file an assignment for future changes.

Jason

Reply via email to