[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