[PATCH] D111282: Don't print uselistorder in --print-changed

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 7 10:37:06 PDT 2021


aeubanks added a comment.

In D111282#3048567 <https://reviews.llvm.org/D111282#3048567>, @ychen wrote:

> I was suggesting this to catch non-determinism issues like the one D64866 <https://reviews.llvm.org/D64866> fixed. Sounds good to me to make it off by default. Any way to make it opt-in? Making it part of API (add a field to PrintPassOptions) would be the best but it could be a follow-up patch with a FIXME added to this patch.

Yeah, adding it as a separate flag sounds good if anybody needs it


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111282



More information about the llvm-commits mailing list