[PATCH] D31239: [WIP] Add Caching of Known Bits in InstCombine

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 22 13:38:41 PDT 2017


rnk added a comment.

We can't just keep passing more analysis pointers throughout our simplification utilities. We might want some wrapper of common, almost globally used analyses that optionally contains DL, TLI, DT, AC, etc.

Long term, I agree with Danny that computing known bits on demand doesn't seem to scale. We might want to just do the analysis up front and see if that works better.


https://reviews.llvm.org/D31239





More information about the llvm-commits mailing list