Code review comment for lp:~ams-codesourcery/gcc-linaro/lp836401-4.6

Revision history for this message
Ulrich Weigand (uweigand) wrote :

This looks good as far as it goes, but there seem to be ongoing discussions on the mailing list regarding the mainline submission of this patch.

Since those discussion are about *further* restricting that test, I'd be fine with the patch submitted here to go in now as is, and then be updated with whatever gets committed upstream in the end.

In the alternative, you might want to wait until the mainline patch is accepted and then check it in to Linaro GCC as well -- feel free to choose either approach.

review: Approve

« Back to merge proposal