[llvm] Fix getParent / getNodeBaseParent to return pointer to const (#94224) (PR #96053)
Stephen Tozer via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 19 04:03:01 PDT 2024
https://github.com/SLTozer approved this pull request.
> Ideally, I'd prefer `ParentPtrTy` to be `ParentTy` and used as `ParentTy*` throughout but it's a much larger change.
I just commented exactly that on the original PR - I originally passed around `ParentPtrTy` to preserve the possibility of a `void*` parent pointer since it seemed like a 0-cost decision that could possibly be useful in future, but since it now results in messier template logic I'm fine to drop it.
This patch LGTM, so if you need to merge it to unblock something downstream then go ahead; if it's not so urgent, then I'll upload a patch that fixes it as described above instead.
https://github.com/llvm/llvm-project/pull/96053
More information about the llvm-commits
mailing list