[PATCH] D75634: [ConstantFolding] Accept FoldedOps cache as parameter

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 15 08:33:09 PDT 2020


nikic abandoned this revision.
nikic added a comment.

Going to close this one, as I actually see a 0.2% regression with it (http://llvm-compile-time-tracker.com/compare.php?from=b236b4cb430674a98917ad1b2fa9f7f9838e66f1&to=a0dcbfc3080e6dd91c7b4253dd83e9eea4eefd03&stat=instructions). Presumably, for the cases where we don't have a lot of unfoldable constant expressions (which is the common case), we end up not actually benefiting from the cache, but still paying (a bit) for it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75634





More information about the llvm-commits mailing list