[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