[PATCH][ARM] Label offset crashes llvm integrated assembler
Jiangning Liu
liujiangning1 at gmail.com
Thu Mar 27 00:50:13 PDT 2014
committed as r204898.
2014-03-21 18:13 GMT+08:00 Renato Golin <renato.golin at linaro.org>:
> On 21 March 2014 03:00, Jiangning Liu <liujiangning1 at gmail.com> wrote:
> > Hi Saleem,
> >
> > Your patch r203762 is reverted by r204427, because the fix is even worse
> > than before, and it introduced silent failure.
> >
> > BTW, should we follow review process before committing any code to trunk?
>
> Hi Jiangning, Saleem,
>
> Yes, I agree we should still review IAS patches before commit, as they
> tend to wash with the baby water on my stream and the IAS is not yet
> stable or well tested to warrant quick changes.
>
> cheers,
> --renato
>
--
Thanks,
-Jiangning
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140327/21097821/attachment.html>
More information about the llvm-commits
mailing list