Skip to content

[ROCm] rename HIP_HCC_FLAGS to HIP_CLANG_FLAGS#50917

Closed
jeffdaily wants to merge 1 commit intopytorch:masterfrom
ROCm:rocm_upstream_hip_clang_flags
Closed

[ROCm] rename HIP_HCC_FLAGS to HIP_CLANG_FLAGS#50917
jeffdaily wants to merge 1 commit intopytorch:masterfrom
ROCm:rocm_upstream_hip_clang_flags

Conversation

@jeffdaily
Copy link
Copy Markdown
Collaborator

ROCm 3.5 replaced hcc with hip-clang and deprecated HIP_HCC_FLAGS.
HIP_CLANG_FLAGS should be used moving forward. HIP_HCC_FLAGS will
be removed soon.

@jeffdaily jeffdaily added the module: rocm AMD GPU support for Pytorch label Jan 21, 2021
@jeffdaily jeffdaily requested a review from walterddr January 21, 2021 22:58
@facebook-github-bot
Copy link
Copy Markdown
Contributor

facebook-github-bot commented Jan 21, 2021

💊 CI failures summary and remediations

As of commit 06718cb (more details on the Dr. CI page):


  • 1/1 failures possibly* introduced in this PR
    • 1/1 non-CircleCI failure(s)

This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

ROCm 3.5 replaced hcc with hip-clang and deprecated HIP_HCC_FLAGS.
HIP_CLANG_FLAGS should be used moving forward. HIP_HCC_FLAGS will
be removed soon.
@ejguan ejguan added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Jan 21, 2021
@jeffdaily jeffdaily force-pushed the rocm_upstream_hip_clang_flags branch from b1a2d4b to 06718cb Compare January 21, 2021 23:05
Copy link
Copy Markdown
Contributor

@walterddr walterddr left a comment

Choose a reason for hiding this comment

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

Lgtm. Will import and merge once CI show green

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@walterddr has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 22, 2021

Codecov Report

Merging #50917 (06718cb) into master (1cce4c5) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #50917      +/-   ##
==========================================
- Coverage   81.02%   81.02%   -0.01%     
==========================================
  Files        1916     1916              
  Lines      209349   209349              
==========================================
- Hits       169628   169619       -9     
- Misses      39721    39730       +9     

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@walterddr merged this pull request in b2e5617.

jeffdaily added a commit to ROCm/pytorch that referenced this pull request Apr 27, 2021
Summary:
ROCm 3.5 replaced hcc with hip-clang and deprecated HIP_HCC_FLAGS.
HIP_CLANG_FLAGS should be used moving forward. HIP_HCC_FLAGS will
be removed soon.

Pull Request resolved: pytorch#50917

Reviewed By: ejguan

Differential Revision: D26008094

Pulled By: walterddr

fbshipit-source-id: cfec4f96fbd9bd338834a841c37267f6a4703cab
@jithunnair-amd jithunnair-amd deleted the rocm_upstream_hip_clang_flags branch May 12, 2021 16:41
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
ROCm 3.5 replaced hcc with hip-clang and deprecated HIP_HCC_FLAGS.
HIP_CLANG_FLAGS should be used moving forward. HIP_HCC_FLAGS will
be removed soon.

Pull Request resolved: pytorch#50917

Reviewed By: ejguan

Differential Revision: D26008094

Pulled By: walterddr

fbshipit-source-id: cfec4f96fbd9bd338834a841c37267f6a4703cab
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed Merged module: rocm AMD GPU support for Pytorch open source triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants