[PATCH] D82730: [SimplifyCFG] Merge identical basic blocks (WIP)

Aditya Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 30 11:56:38 PDT 2020


hiraditya added a comment.

SimplifyCFG doesn't look like a good place to put this as it is called multiple times and impacts compile time significantly.
Agreed with @jfb that having hasher as a separate pass which other passes can consume. @vish99 may have more ideas on hashing.
Hashing can be expensive and we could use clever tricks to make it cheaper. D52896 <https://reviews.llvm.org/D52896> could help, happy to provide more input.


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

https://reviews.llvm.org/D82730





More information about the llvm-commits mailing list