Skip to content

Commit 92b761a

Browse files
stanleytsang-amdumfranzwamd-garydengeidenyoshidaamd-jnovotny
authored
Merge back 6.3 hotfixes (#588)
* Remove website URL from comments (#542) Referencing or using code from some websites is prohibited in this repository. This change removes an informational reference in the comments. * Add gfx1151 target (#543) (#546) Co-authored-by: Stanley Tsang <[email protected]> Co-authored-by: Eiden Yoshida <[email protected]> * Move data type support file along with index and ToC changes (#548) (#549) * Convert change log to new format (#564) * Fixed memory leaks in rocrand_tests (#557) * added hipFree to test_rocrand_cpp_basic * fixed memory leak for test_rocrand_config_dispatch * fixed a memory leak in test_utils * changed createGraph to createAndLaunchGraph, as well as fixed stream capture order * changed default boolean (kaunchGraph, sync) to be true in createAndLaunchGraph * added back missing end stream capture * reformated curlys for consistency * removed createAndLaunchGraph inside resetGraphHelper * Update changelog release headers (#568) (#575) * Update changelog release headers * Small correction (cherry picked from commit c7da202) * Add readme and metadata info and clarify CUDA references (#567) (#576) (cherry picked from commit 81d9e58) * added gfx12 and gfx1151 to default gpu list * updated changelog * Update CHANGELOG.md Co-authored-by: Jeffrey Novotny <[email protected]> * Update CHANGELOG.md Co-authored-by: Jeffrey Novotny <[email protected]> * Remove gfx940,gfx941 targets (#580) * Remove unreleased --------- Co-authored-by: Wayne Franz <[email protected]> Co-authored-by: amd-garydeng <[email protected]> Co-authored-by: Eiden Yoshida <[email protected]> Co-authored-by: Jeffrey Novotny <[email protected]> Co-authored-by: Di Nguyen <[email protected]> Co-authored-by: Val Movsik <[email protected]>
1 parent 7f0ee87 commit 92b761a

File tree

3 files changed

+11
-7
lines changed

3 files changed

+11
-7
lines changed

CHANGELOG.md

+5-4
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
Documentation for rocRAND is available at
44
[https://rocm.docs.amd.com/projects/rocRAND/en/latest/](https://rocm.docs.amd.com/projects/rocRAND/en/latest/)
55

6-
## (Unreleased) rocRAND 3.3.0 for ROCm 6.4
6+
## rocRAND 3.3.0 for ROCm 6.4
77

88
### Added
99

@@ -29,14 +29,15 @@ Documentation for rocRAND is available at
2929

3030
### Changed
3131

32+
* Updated the default value for the `-a` argument from `rmake.py` to `gfx906:xnack-,gfx1030,gfx1100,gfx1101,gfx1102,gfx1151,gfx1200,gfx1201`.
3233
* `rocrand_discrete` for MTGP32, LFSR113 and ThreeFry generators now uses the alias method, which is faster than binary search in CDF.
3334

3435
## rocRAND 3.1.1 for ROCm 6.2.4
3536

3637
## Fixes
3738

38-
* Fixed " unknown extension ?>" issue in scripts/config-tuning/select_best_config.py
39-
when using python version thats older than 3.11
39+
* Fixed an issue in `rmake.py` where the list storing cmake options would contain individual characters instead of a full string of options.
40+
* Fixed " unknown extension ?>" issue in scripts/config-tuning/select_best_config.py when using python version thats older than 3.11
4041
* Fixed low random sequence quality of `ROCRAND_RNG_PSEUDO_THREEFRY2_64_20` and `ROCRAND_RNG_PSEUDO_THREEFRY4_64_20`.
4142

4243
## rocRAND 3.1.0 for ROCm 6.2.0
@@ -113,7 +114,7 @@ Documentation for rocRAND is available at
113114
### Known issues
114115
- SOBOL64 and SCRAMBLED_SOBOL64 generate poisson-distributed `unsigned long long int` numbers instead of `unsigned int`. This will be fixed in the next major release.
115116

116-
## (Unreleased) rocRAND-3.0.0 for ROCm 6.0.0
117+
## rocRAND-3.0.0 for ROCm 6.0.0
117118

118119
### Additions
119120

CMakeLists.txt

+1-1
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ if(GPU_TARGETS STREQUAL "all")
9797
if(BUILD_ADDRESS_SANITIZER)
9898
# ASAN builds require xnack
9999
rocm_check_target_ids(DEFAULT_AMDGPU_TARGETS
100-
TARGETS "gfx908:xnack+;gfx90a:xnack+;gfx940:xnack+;gfx941:xnack+;gfx942:xnack+"
100+
TARGETS "gfx908:xnack+;gfx90a:xnack+;gfx942:xnack+"
101101
)
102102
else()
103103
rocm_check_target_ids(DEFAULT_AMDGPU_TARGETS

rmake.py

+5-2
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,9 @@ def parse_args():
2020
parser = argparse.ArgumentParser(description="""
2121
Checks build arguments
2222
""")
23+
24+
default_gpus = 'gfx906:xnack-,gfx1030,gfx1100,gfx1101,gfx1102,gfx1151,gfx1200,gfx1201'
25+
2326
parser.add_argument('-g', '--debug', required=False, default=False, action='store_true',
2427
help='Generate Debug build (default: False)')
2528
parser.add_argument( '--build_dir', type=str, required=False, default="build",
@@ -37,7 +40,7 @@ def parse_args():
3740
help='Install after build (default: False)')
3841
parser.add_argument( '--cmake-darg', required=False, dest='cmake_dargs', action='append', default=[],
3942
help='List of additional cmake defines for builds (e.g. CMAKE_CXX_COMPILER_LAUNCHER=ccache)')
40-
parser.add_argument('-a', '--architecture', dest='gpu_architecture', required=False, default="gfx906;gfx1030;gfx1100;gfx1101;gfx1102", #:sramecc+:xnack-" ) #gfx1030" ) #gfx906" ) # gfx1030" )
43+
parser.add_argument('-a', '--architecture', dest='gpu_architecture', required=False, default=default_gpus, #:sramecc+:xnack-" ) #gfx1030" ) #gfx906" ) # gfx1030" )
4144
help='Set GPU architectures, e.g. all, gfx000, gfx803, gfx906:xnack-;gfx1030;gfx1100 (optional, default: all)')
4245
parser.add_argument('-v', '--verbose', required=False, default=False, action='store_true',
4346
help='Verbose build (default: False)')
@@ -111,7 +114,7 @@ def config_cmd():
111114
else:
112115
cmake_executable = "cmake"
113116
toolchain = "toolchain-linux.cmake"
114-
cmake_platform_opts = f"-DROCM_DIR:PATH={rocm_path} -DCPACK_PACKAGING_INSTALL_PREFIX={rocm_path}"
117+
cmake_platform_opts = [f"-DROCM_DIR:PATH={rocm_path}", f"-DCPACK_PACKAGING_INSTALL_PREFIX={rocm_path}"]
115118

116119
tools = f"-DCMAKE_TOOLCHAIN_FILE={toolchain}"
117120
cmake_options.append( tools )

0 commit comments

Comments
 (0)