Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix gcc build on NVCC path#1661

Merged
mangupta merged 3 commits into
ROCm:masterfrom
gargrahul:gcc_nvcc_path
Nov 18, 2019
Merged

Fix gcc build on NVCC path#1661
mangupta merged 3 commits into
ROCm:masterfrom
gargrahul:gcc_nvcc_path

Conversation

@gargrahul
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

@mangupta mangupta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mangupta mangupta added the pr:needs_updates PR initially approved. But needs rework label Nov 18, 2019
@mangupta mangupta merged commit e39d749 into ROCm:master Nov 18, 2019
ammallya pushed a commit that referenced this pull request Nov 26, 2025
…on cuda (#1152) & HSA Table 8 API to maintian ABI backward compatibility of rocprofiler-sdk (#1640)

* SWDEV-557828 - fix hip-tests on cuda (#1152)

Co-authored-by: Rahul Manocha <[email protected]>

* [ROCProfiler-SDK][ROCR] HSA New API changes for HSA_AMD_EXT_API_TABLE_STEP_VERSION 8 (#1182)

* add new hsa ext api for version 8.

* use fmt instead of ostream.

* override rccl from therock

* Update rocprofiler-sdk-continuous_integration.yml

* Update rocprofiler-sdk-continuous_integration.yml

* Update rocprofiler-sdk-continuous_integration.yml

* enable rocr-build

* format

* disable att consecutive-kernels tests.

* Enable ROCR build in code coverage workflow

---------

Co-authored-by: Venkateshwar Reddy Kandula <[email protected]>

* misc.

* [ROCProfiler-SDK] Remove 'gfx900' and 'gfx940' from GPU targets (#1661)

* [ROCProfiler-SDK] Remove 'gfx900' and 'gfx940' from GPU targets

* Remove unsupported GPU targets from workflow

* Remove gfx900 and gfx940 from GPU targets

---------

Co-authored-by: Rahul Manocha <[email protected]>
Co-authored-by: Rahul Manocha <[email protected]>
Co-authored-by: Venkateshwar Reddy Kandula <[email protected]>
Co-authored-by: Ammar ELWazir <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:needs_updates PR initially approved. But needs rework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants