[PATCH] D142330: [AssumptionCache] caches @llvm.experimental.guard's

[Max Kazantsev via Phabricator via llvm-commits]

mkazantsev added a comment.

This chance has caused us a huge compile time degradation. @caojoshua I wonder, are you using guards for some reason? And why have you chosen to use it instead of new representation based on https://llvm.org/docs/LangRef.html#llvm-experimental-widenable-condition-intrinsic?

I think that experimental.guard wasn't our best design decision, and we in Azul are trying to completely get rid of them (it's moving slowly with pain, but it's moving). The reason of this is the requirement to duplicate every single branch-based transform for guards intrinsic, which is practically impossible. Unless there are strong reasons to stick with them, I would strongly recommend to use branch-based form that doesn't have this problem.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142330