[PATCH] D110589: [IntelJITListener] Generalize JIT listener test

Lang Hames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 27 16:12:44 PDT 2021


lhames accepted this revision.
lhames added a comment.
This revision is now accepted and ready to land.

If the order is expected to be stable within a release then re-ordering to match the new expected order seems like a good option, since it alert you to any unexpected changes. I'm ok with this approach too though.

Thanks very much for looking into this Andy!


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

https://reviews.llvm.org/D110589



More information about the llvm-commits mailing list