[PATCH] D143468: [CMake] Remove custom ccache CMake logic

Wed Feb 15 23:23:06 PST 2023

phosek added a comment.

In D143468#4128414 <https://reviews.llvm.org/D143468#4128414>, @thieta wrote:

> @jrtc27 raises a good point - we have discussed deprecating the option but made it a hard error. Should we instead make it a soft error for now and a hard error later on?
>
> Honestly, I was not expecting this option to be used as much as it seems like it is. So maybe it makes sense to make it a soft error to start.
>
> Any thoughts on this so I can make the appropriate changes?

I'd be fine starting with a warning and make it an error after the LLVM 17 branch point.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143468