[llvm-branch-commits] [llvm] AMDGPU: Start using subarch in attributor instead of subtarget (PR #212207)

[via llvm-branch-commits]

github-actions[bot] wrote:

<!--PREMERGE ADVISOR COMMENT: Linux-->
# :penguin: Linux x64 Test Results

* 127530 tests passed
* 2444 tests skipped

All executed tests passed, but another part of the build **failed**. Click on a failure below to see the details.

<details>
<summary>tools/clang/unittests/Basic/CMakeFiles/BasicTests.dir/OffloadArchTest.cpp.o</summary>

```
FAILED: tools/clang/unittests/Basic/CMakeFiles/BasicTests.dir/OffloadArchTest.cpp.o
sccache /opt/llvm/bin/clang++ -DLLVM_BUILD_STATIC -D_DEBUG -D_GLIBCXX_ASSERTIONS -D_GLIBCXX_USE_CXX11_ABI=1 -D_GNU_SOURCE -D_LIBCPP_HARDENING_MODE=_LIBCPP_HARDENING_MODE_EXTENSIVE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -I/home/gha/actions-runner/_work/llvm-project/llvm-project/build/tools/clang/unittests/Basic -I/home/gha/actions-runner/_work/llvm-project/llvm-project/clang/unittests/Basic -I/home/gha/actions-runner/_work/llvm-project/llvm-project/clang/include -I/home/gha/actions-runner/_work/llvm-project/llvm-project/build/tools/clang/include -I/home/gha/actions-runner/_work/llvm-project/llvm-project/build/include -I/home/gha/actions-runner/_work/llvm-project/llvm-project/llvm/include -I/home/gha/actions-runner/_work/llvm-project/llvm-project/third-party/unittest/googletest/include -I/home/gha/actions-runner/_work/llvm-project/llvm-project/third-party/unittest/googlemock/include -gmlt -fPIC -fno-semantic-interposition -fvisibility-inlines-hidden -Werror -Werror=date-time -Werror=unguarded-availability-new -Wall -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -pedantic -Wno-long-long -Wc++98-compat-extra-semi -Wimplicit-fallthrough -Wcovered-switch-default -Wno-noexcept-type -Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Wsuggest-override -Wstring-conversion -Wno-pass-failed -Wmisleading-indentation -Wctad-maybe-unsupported -fdiagnostics-color -ffunction-sections -fdata-sections -fno-common -Woverloaded-virtual -Wno-nested-anon-types -O3 -DNDEBUG -std=c++17 -UNDEBUG -Wno-variadic-macros -Wno-gnu-zero-variadic-macro-arguments -fno-exceptions -funwind-tables -fno-rtti -Wno-suggest-override -MD -MT tools/clang/unittests/Basic/CMakeFiles/BasicTests.dir/OffloadArchTest.cpp.o -MF tools/clang/unittests/Basic/CMakeFiles/BasicTests.dir/OffloadArchTest.cpp.o.d -o tools/clang/unittests/Basic/CMakeFiles/BasicTests.dir/OffloadArchTest.cpp.o -c /home/gha/actions-runner/_work/llvm-project/llvm-project/clang/unittests/Basic/OffloadArchTest.cpp
/home/gha/actions-runner/_work/llvm-project/llvm-project/clang/unittests/Basic/OffloadArchTest.cpp:77:46: error: no member named 'GFX600' in 'clang::OffloadArch'
77 |   for (int I = static_cast<int>(OffloadArch::GFX600);
|                                              ^~~~~~
/home/gha/actions-runner/_work/llvm-project/llvm-project/clang/unittests/Basic/OffloadArchTest.cpp:78:42: error: no member named 'AMDGCNSPIRV' in 'clang::OffloadArch'
78 |        I < static_cast<int>(OffloadArch::AMDGCNSPIRV); ++I) {
|                                          ^~~~~~~~~~~
/home/gha/actions-runner/_work/llvm-project/llvm-project/clang/unittests/Basic/OffloadArchTest.cpp:79:24: error: no matching conversion for static_cast from 'int' to 'OffloadArch'
79 |     OffloadArch Arch = static_cast<OffloadArch>(I);
|                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/gha/actions-runner/_work/llvm-project/llvm-project/clang/include/clang/Basic/OffloadArch.h:32:7: note: candidate constructor (the implicit copy constructor) not viable: no known conversion from 'int' to 'const OffloadArch' for 1st argument
32 | class OffloadArch {
|       ^~~~~~~~~~~
/home/gha/actions-runner/_work/llvm-project/llvm-project/clang/include/clang/Basic/OffloadArch.h:32:7: note: candidate constructor (the implicit move constructor) not viable: no known conversion from 'int' to 'OffloadArch' for 1st argument
32 | class OffloadArch {
|       ^~~~~~~~~~~
/home/gha/actions-runner/_work/llvm-project/llvm-project/clang/include/clang/Basic/OffloadArch.h:59:13: note: candidate constructor not viable: requires 0 arguments, but 1 was provided
59 |   constexpr OffloadArch() = default;
|             ^
/home/gha/actions-runner/_work/llvm-project/llvm-project/clang/include/clang/Basic/OffloadArch.h:56:13: note: candidate constructor not viable: requires 2 arguments, but 1 was provided
56 |   constexpr OffloadArch(TargetArch V, uint32_t Kind) : Kind(Kind), V(V) {}
|             ^           ~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/gha/actions-runner/_work/llvm-project/llvm-project/clang/unittests/Basic/OffloadArchTest.cpp:80:17: error: use of undeclared identifier 'IsAMDOffloadArch'; did you mean 'OffloadArch'?
80 |     EXPECT_TRUE(IsAMDOffloadArch(Arch));
|                 ^~~~~~~~~~~~~~~~
/home/gha/actions-runner/_work/llvm-project/llvm-project/clang/include/clang/Basic/OffloadArch.h:32:7: note: 'OffloadArch' declared here
32 | class OffloadArch {
|       ^
/home/gha/actions-runner/_work/llvm-project/llvm-project/clang/unittests/Basic/OffloadArchTest.cpp:89:48: error: no member named 'AMDGCNSPIRV' in 'clang::OffloadArch'
89 |   EXPECT_EQ(getOffloadArchSubArch(OffloadArch::AMDGCNSPIRV),
|                                                ^~~~~~~~~~~
5 errors generated.
```
</details>

If these failures are unrelated to your changes (for example tests are broken or flaky at HEAD), please open an issue at https://github.com/llvm/llvm-project/issues and add the `infrastructure` label.

https://github.com/llvm/llvm-project/pull/212207