[PATCH] D29115: [AMDGPU] Bump up n_type for metadata v2

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 24 22:04:23 PST 2017


arsenm added a comment.

In https://reviews.llvm.org/D29115#655900, @kzhuravl wrote:

> In https://reviews.llvm.org/D29115#655884, @arsenm wrote:
>
> > Needs a test update?
>
>
> No. But we can add a check lines to metadata tests to check for n_type?




In https://reviews.llvm.org/D29115#655900, @kzhuravl wrote:

> In https://reviews.llvm.org/D29115#655884, @arsenm wrote:
>
> > Needs a test update?
>
>
> No. But we can add a check lines to metadata tests to check for n_type?


Yes, I think it's surprising there wasn't one before


https://reviews.llvm.org/D29115





More information about the llvm-commits mailing list