[PATCH] D39560: [AMDGPU] Convert lit tests to new addr space mapping, part 1

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 5 15:52:43 PST 2017


arsenm added a comment.

The script isn't that big, so in the past the policy has been to just put the body of the script literally in the commit message rather than adding a temporary file that will be forgotten.

This is adding the amdgiz environment to all the tests, which is not what I expected. I expected that to be removed and to change the default to the new mapping thus removing the need for the triple changes in all of these tests. Is your plan to do that as a separate step after?


https://reviews.llvm.org/D39560





More information about the llvm-commits mailing list