[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