Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[ATD] Updated ATD & ATD_ASO_full.patch #1167

Open
wants to merge 1 commit into
base: aomp-dev
Choose a base branch
from
Open

[ATD] Updated ATD & ATD_ASO_full.patch #1167

wants to merge 1 commit into from

Conversation

dpalermo
Copy link
Contributor 

amd-staging      4358e7907409
trunk            fa4d1860d20a
trunk-atd        67697c6bab77

@dpalermo
[ATD] Updated ATD & ATD_ASO_full.patch
81e6b91 
    amd-staging      4358e7907409
    trunk            fa4d1860d20a
    trunk-atd        67697c6bab77
@gregrodgers
Copy link
Contributor

Dan, no reason to wait for PR review of these two patch files. Whenever, you have a more up-to-date version of either ATD_full,patch or ATD_ASO_full.patch, please push this into aomp aomp-dev branch . Yesterday and today I pushed updates that bring these up to date. So I think we can delete this PR.

That said, there is a PR to atomate applying ATD_ASO_full.patch #1169 to the aomp build scripts that use llvm-project. That should be reviewed and merged if only for awareness that this is now automated (if desired) in the build of aomp.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ronlieb ronlieb ronlieb approved these changes

@estewart08 estewart08 Awaiting requested review from estewart08 estewart08 is a code owner

@gregrodgers gregrodgers Awaiting requested review from gregrodgers gregrodgers is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dpalermo @gregrodgers @ronlieb