[llvm] [SelectionDAG] Stop storing EVTs in a function scoped static std::set. (PR #118715)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 4 16:30:12 PST 2024


topperc wrote:

> Is there some reason you're switching to FoldingSet, instead of just sticking with std::set? Not sure what the performance of FoldingSet is like, but any performance difference is probably related to that switch, not whether the std::set is a global.

Only because the FoldingSet already existed for the multiple element case so it was easy to reuse. I'll try with a std::set.

https://github.com/llvm/llvm-project/pull/118715


More information about the llvm-commits mailing list