Skip to content

[AMDGPU] Simplify definition of AddrSpaces. NFC. #141030

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 22, 2025

Conversation

jayfoad
Copy link
Contributor

@jayfoad jayfoad commented May 22, 2025

No description provided.

@llvmbot
Copy link
Member

llvmbot commented May 22, 2025

@llvm/pr-subscribers-backend-amdgpu

Author: Jay Foad (jayfoad)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/141030.diff

1 Files Affected:

  • (modified) llvm/lib/Target/AMDGPU/AMDGPUInstructions.td (+1-3)
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUInstructions.td b/llvm/lib/Target/AMDGPU/AMDGPUInstructions.td
index 78a92d85cfd8e..18a948d68e97b 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUInstructions.td
+++ b/llvm/lib/Target/AMDGPU/AMDGPUInstructions.td
@@ -11,7 +11,7 @@
 //
 //===----------------------------------------------------------------------===//
 
-class AddressSpacesImpl {
+def AddrSpaces {
   int Flat = 0;
   int Global = 1;
   int Region = 2;
@@ -21,8 +21,6 @@ class AddressSpacesImpl {
   int Constant32Bit = 6;
 }
 
-def AddrSpaces : AddressSpacesImpl;
-
 
 class AMDGPUInst <dag outs, dag ins, string asm = "",
   list<dag> pattern = []> : Instruction {

@jayfoad jayfoad merged commit e1db811 into llvm:main May 22, 2025
8 of 13 checks passed
@jayfoad jayfoad deleted the simplify-addrspaces branch May 22, 2025 10:14
@llvm-ci
Copy link
Collaborator

llvm-ci commented May 22, 2025

LLVM Buildbot has detected a new failure on builder lldb-aarch64-ubuntu running on linaro-lldb-aarch64-ubuntu while building llvm at step 6 "test".

Full details are available at: https://lab.llvm.org/buildbot/#/builders/59/builds/18110

Here is the relevant piece of the build log for the reference
Step 6 (test) failure: build (failure)
...
PASS: lldb-unit :: ValueObject/./LLDBValueObjectTests/8/11 (2175 of 2184)
PASS: lldb-unit :: ValueObject/./LLDBValueObjectTests/9/11 (2176 of 2184)
PASS: lldb-unit :: tools/lldb-server/tests/./LLDBServerTests/0/2 (2177 of 2184)
PASS: lldb-unit :: tools/lldb-server/tests/./LLDBServerTests/1/2 (2178 of 2184)
PASS: lldb-unit :: Utility/./UtilityTests/4/9 (2179 of 2184)
PASS: lldb-unit :: Target/./TargetTests/11/14 (2180 of 2184)
PASS: lldb-unit :: Host/./HostTests/8/13 (2181 of 2184)
PASS: lldb-unit :: Host/./HostTests/0/13 (2182 of 2184)
PASS: lldb-unit :: Process/gdb-remote/./ProcessGdbRemoteTests/8/9 (2183 of 2184)
TIMEOUT: lldb-api :: functionalities/gdb_remote_client/TestGDBRemoteClient.py (2184 of 2184)
******************** TEST 'lldb-api :: functionalities/gdb_remote_client/TestGDBRemoteClient.py' FAILED ********************
Script:
--
/usr/bin/python3.10 /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/llvm-project/lldb/test/API/dotest.py -u CXXFLAGS -u CFLAGS --env LLVM_LIBS_DIR=/home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/./lib --env LLVM_INCLUDE_DIR=/home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/include --env LLVM_TOOLS_DIR=/home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/./bin --arch aarch64 --build-dir /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/lldb-test-build.noindex --lldb-module-cache-dir /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/lldb-test-build.noindex/module-cache-lldb/lldb-api --clang-module-cache-dir /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/lldb-test-build.noindex/module-cache-clang/lldb-api --executable /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/./bin/lldb --compiler /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/./bin/clang --dsymutil /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/./bin/dsymutil --make /usr/bin/gmake --llvm-tools-dir /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/./bin --lldb-obj-root /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/tools/lldb --lldb-libs-dir /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/./lib /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/llvm-project/lldb/test/API/functionalities/gdb_remote_client -p TestGDBRemoteClient.py
--
Exit Code: -9
Timeout: Reached timeout of 600 seconds

Command Output (stdout):
--
lldb version 21.0.0git (https://github.com/llvm/llvm-project.git revision e1db811341d361dde9767d41fe959efbca1adcbe)
  clang revision e1db811341d361dde9767d41fe959efbca1adcbe
  llvm revision e1db811341d361dde9767d41fe959efbca1adcbe

--

********************
********************
Timed Out Tests (1):
  lldb-api :: functionalities/gdb_remote_client/TestGDBRemoteClient.py


Testing Time: 672.57s

Total Discovered Tests: 3337
  Unsupported      :  519 (15.55%)
  Passed           : 2792 (83.67%)
  Expectedly Failed:   25 (0.75%)
  Timed Out        :    1 (0.03%)
FAILED: tools/lldb/test/CMakeFiles/check-lldb /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/tools/lldb/test/CMakeFiles/check-lldb 
cd /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/tools/lldb/test && /usr/bin/python3.10 /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/./bin/llvm-lit -v /home/tcwg-buildbot/worker/lldb-aarch64-ubuntu/build/tools/lldb/test
ninja: build stopped: subcommand failed.

sivan-shani pushed a commit to sivan-shani/llvm-project that referenced this pull request Jun 3, 2025
ajaden-codes pushed a commit to Jaddyen/llvm-project that referenced this pull request Jun 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants