[PATCH] D16684: [ARM] Emit trap instruction using .inst directive

James Molloy via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 29 02:37:08 PST 2016


Hi,

I'm not happy with the test changes; it was checking that at least the
comment said 'trap' - what happens now if we regress and the .inst is t
even emitted for that testcase?

James
On Fri, 29 Jan 2016 at 10:27, Phabricator via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> This revision was automatically updated to reflect the committed changes.
> Closed by commit rL259182: [ARM] Emit trap instruction using .inst
> directive (authored by alelab01).
>
> Changed prior to commit:
>   http://reviews.llvm.org/D16684?vs=46269&id=46367#toc
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D16684
>
> Files:
>   llvm/trunk/lib/Target/ARM/ARMAsmPrinter.cpp
>   llvm/trunk/test/CodeGen/ARM/debugtrap.ll
>   llvm/trunk/test/CodeGen/ARM/trap.ll
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160129/63a6e7a5/attachment.html>


More information about the llvm-commits mailing list