[PATCH] D61521: [AMDGPU] gfx1010 verifier changes

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 3 10:37:51 PDT 2019


arsenm added a comment.

In D61521#1490105 <https://reviews.llvm.org/D61521#1490105>, @rampitec wrote:

> In D61521#1490093 <https://reviews.llvm.org/D61521#1490093>, @arsenm wrote:
>
> > Should add verifier tests
>
>
> I do not see verifier tests anywhere for our BE in general. We need to add them all at once, i.e. a separate change.


All at once would be worse than adding pieces as they are added


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61521/new/

https://reviews.llvm.org/D61521





More information about the llvm-commits mailing list