[PATCH] D111688: [MachineSink] Compile time improvement for large testcases which has many kill flags We did a experiment and observed dramatic decrease on compilation time which spent on clearing kill flags.

Bing Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 13 19:28:43 PDT 2021


yubing added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineSink.cpp:134
 
-    SparseBitVector<> RegsToClearKillFlags;
+    DenseSet<unsigned> RegsToClearKillFlags;
 
----------------
MatzeB wrote:
> Does `DenseSet<Register>` work too? (if it does not then I'm fine with this).
Thanks for suggestions, DenseSet<Register> works for me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111688



More information about the llvm-commits mailing list