[llvm] [Offload] Fix APU detection for MI300 testing (PR #143026)

via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 5 12:57:38 PDT 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-offload

Author: Joseph Huber (jhuber6)

<details>
<summary>Changes</summary>

Summary:
We have this check when the target is MI300 but it fails if this
environment variable isn't set. Set a default value of '0' if not
present so that will be converted to bool false.


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


1 Files Affected:

- (modified) offload/test/lit.cfg (+1-2) 


``````````diff
diff --git a/offload/test/lit.cfg b/offload/test/lit.cfg
index 0725b56f0f05d..df57be293138c 100644
--- a/offload/test/lit.cfg
+++ b/offload/test/lit.cfg
@@ -36,8 +36,7 @@ if 'HSA_ENABLE_SDMA' in os.environ:
 
 # Architectures like gfx942 may or may not be APUs so an additional environment
 # variable is required as some tests can be APU specific.
-if 'IS_APU' in os.environ:
-    config.environment['IS_APU'] = os.environ['IS_APU']
+config.environment['IS_APU'] = os.environ.get('IS_APU', '0')
 
 # set default environment variables for test
 if 'CHECK_OPENMP_ENV' in os.environ:

``````````

</details>


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


More information about the llvm-commits mailing list