[lldb] [llvm] Modify the localCache API to require an explicit commit on CachedFile… (PR #136121)

via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 24 07:56:21 PDT 2025


dyung wrote:

> The original PR (#115331) was approved and merged, but later reverted due an unstable test. The only difference in this version of the PR was a fix to make the test stable on Windows, so I thought it would be ok to merge by the approval of the original PR. Apologies if I was mistaken on this.

Who approved the original PR? Was the approval done offline? I do not see an approval in the original PR, just some feedback.

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


More information about the llvm-commits mailing list