Skip to content

[Github Action] Enable automatic sync for main branch from llvm-project to llvm #2904

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.

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

Merged
merged 5 commits into from
Dec 21, 2020

Conversation

hanzhan1
Copy link
Contributor

Create sync-main.yml to automatic sync for main branch from llvm-project to llvm

Create sync-main.yml to automatic sync for main branch from llvm-project to llvm
@hanzhan1 hanzhan1 requested a review from DoyleLi December 18, 2020 03:08
@hanzhan1 hanzhan1 requested a review from bader as a code owner December 18, 2020 03:08
DoyleLi
DoyleLi previously approved these changes Dec 18, 2020
Copy link
Contributor

@DoyleLi DoyleLi left a comment

Choose a reason for hiding this comment

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

LGTM.

@hanzhan1 hanzhan1 requested a review from tfzhu December 18, 2020 05:25
tfzhu
tfzhu previously approved these changes Dec 18, 2020
bader
bader previously approved these changes Dec 18, 2020
Co-authored-by: Alexey Bader <alexey.bader@intel.com>
@hanzhan1 hanzhan1 dismissed stale reviews from bader and tfzhu via 4dd0189 December 21, 2020 01:14
Co-authored-by: Alexey Bader <alexey.bader@intel.com>
@hanzhan1 hanzhan1 requested a review from bader December 21, 2020 01:15
@hanzhan1
Copy link
Contributor Author

Hi @bader, I have revised the script as suggested, could you please help to code review and merge the pull request? Thanks!

@bader bader merged commit 4173f2e into intel:sycl Dec 21, 2020
@bader
Copy link
Contributor

bader commented Dec 21, 2020

It fails with invalid cron attribute "/10 * * * *"

See https://github.com/intel/llvm/actions/runs/435454887

@hanzhan1, please, test the fix in your private fork before opening new PR.

alexbatashev pushed a commit to alexbatashev/llvm that referenced this pull request Dec 21, 2020
* upstream/sycl: (616 commits)
  [SYCL][L0] Implement robust error handling in level_zero plugin (intel#2870)
  [SYCL][NFC] Code clean up (phase 5) revealed by self build. (intel#2907)
  [Driver][NFC] Remove unused variable (intel#2908)
  [Github Action] Enable automatic sync for main branch from llvm-project to llvm (intel#2904)
  [ESIMD][NFC] Remove unnecessary macro checks (intel#2900)
  [SYCL] Fix handling of multiple usages of composite spec constants (intel#2894)
  [SYCL] Adjust parallel-for range global size to improve group size selection (intel#2703)
  [SYCL] Add template parameter support for no_global_work_offset attribute (intel#2839)
  [SYCL] Support LLVM FP intrinsic in llvm-spirv and FE (intel#2880)
  [SYCL]Link Libm FP64 SYCL device library by default (intel#2892)
  [SYCL][NFC] Code clean up (phase 4) revealed by self build. (intel#2878)
  [SYCL][NFC] Code clean up (phase 3) revealed by self build. (intel#2865)
  [SYCL] Fix backend selection for SYCL_DEVICE_TYPE=* (intel#2890)
  [SYCL] Fix spec constants support in integration header (intel#2896)
  [Driver] Update unbundling of offload libraries to use archive type (intel#2883)
  [SYCL][NFC] Clang format SYCL.cpp (intel#2891)
  [CODEOWNERS] Add code owners for DPC++ tools (intel#2884)
  [XPTIFW] Enable in-tree builds (intel#2849)
  [SYCL] Don't dump IR and dot files by default in the LowerWGScope pass (intel#2887)
  [SYCL] Use llvm-link's only-needed option to link device libs (intel#2783)
  ...
hanzhan1 added a commit that referenced this pull request Dec 22, 2020
# pushing to the repository. Otherwise, the default github actions token
# is used.
persist-credentials: false
fetch-depth: 0
Copy link
Contributor

Choose a reason for hiding this comment

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

I just noticed that we fetch all tags and branches. @hanzhan1, is it really needed?
I would expect that we need to fetch the HEAD of main branch only. Right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants