[Openmp-commits] [PATCH] D103058: [libomptarget][nfc] Move hostcall required test to rtl

Jon Chesterfield via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue May 25 15:16:43 PDT 2021


JonChesterfield added a comment.

Thanks for pointing it out! I was under the impression that the contents of 'Summary' as shown in Phabricator turns into the commit message.

Looks like the commit message I'm creating is the title from phabricator, then a newline, then the summary, so the workflow change is probably to write less in the Summary field.

I'm committing a patch by:

  git checkout main && git reset --hard origin/main && git pull # clean slate
  arc patch D1000
  git checkout main
  git cherry-pick D1000
  git log -n2 # eyeball it
  git push

combined with creating patches with arc diff, I think the failure mode must be in my use of arcanist.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103058/new/

https://reviews.llvm.org/D103058



More information about the Openmp-commits mailing list