[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