[Openmp-commits] [PATCH] D95530: [libomptarget] Load images in order of registration

Tim Cramer via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed Feb 17 03:59:29 PST 2021


tcramer added a comment.

> I'd personally split this, because annoying though it is to split a change, the whole thing is likely to be reverted if part of it turns out to be faulty.

Since you already accepted this patch, I guess it is ok if we leave it in one patch? If the whole thing has to be reverted it is ok for us. In that case we would fix it with an additonal / splitted patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95530



More information about the Openmp-commits mailing list