[PATCH] D99977: [GVN] Clear ICF cache when we simplify a value

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 8 00:49:21 PDT 2021


nikic added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/GVN.cpp:2174
+      // For example, devirtualization to a willreturn function.
+      ICF->clear();
       Changed = true;
----------------
It would be better to extend the IPT interface to allow more fine grained invalidation here. At least to only invalidate the blocks where the user is the current special instruction.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99977



More information about the llvm-commits mailing list