[PATCH] D99205: Add jump-threading optimization for deterministic finite automata

Diogo N. Sampaio via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 27 04:11:01 PDT 2021


dnsampaio added a comment.

I've finally got time to test this.
For our downstream arch we're seeing gains up to 27%.
Very good, thanks for working on this.



================
Comment at: llvm/lib/Transforms/Scalar/DFAJumpThreading.cpp:363-366
+inline raw_ostream &operator<<(raw_ostream &OS, const ThreadingPath &TPath) {
+  TPath.print(OS);
+  return OS;
+}
----------------
In release mode I get warnings this is not used (we use -Werror):
llvm/lib/Transforms/Scalar/DFAJumpThreading.cpp:364:21: error: unused function 'operator<<' [-Werror,-Wunused-function]

Perhaps put it around #ifndef NDEBUG ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99205



More information about the llvm-commits mailing list