[llvm] [memprof] Introduce readMemProf (NFC) (PR #88095)

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 9 11:36:45 PDT 2024


teresajohnson wrote:

> > difficult to reason about and update consistently.
> 
> The same rationale motivated my suggestion. By separating out the implementation for each version, we don't have to update them at all and simply remove them as they age out. Wouldn't that be easier to reason about as opposed to more complex control flow?

I guess it depends how how much we expect to diverge in future versions. If you think this is going to be easier I am not opposed.

https://github.com/llvm/llvm-project/pull/88095


More information about the llvm-commits mailing list