[all-commits] [llvm/llvm-project] c014db: [ADT] Remove the const variant of LookupBucketFor ...

[Kazu Hirata via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c014db47ca298ad7a0f52a57c3bfc2a9914371b2
      https://github.com/llvm/llvm-project/commit/c014db47ca298ad7a0f52a57c3bfc2a9914371b2
  Author: Kazu Hirata <kazu at google.com>
  Date:   2024-09-06 (Fri, 06 Sep 2024)

  Changed paths:
    M llvm/include/llvm/ADT/DenseMap.h

  Log Message:
  -----------
  [ADT] Remove the const variant of LookupBucketFor in DenseMap (#107608)

DenseMap has const and non-const variants of LookupBucketFor to find a
bucket for insertion purposes.  Now that queries (e.g. find, contains,
and erase) have been migrated to use doFind instead of
LookupBucketFor, nobody uses the const variant of LookupBucketFor.
This patch removes the const variant.

As far as the logistics go, the non-const variant calls the const
variant with a couple of const_cast, so this patch repurposes the
const variant as the non-const one while removing the original
non-const variant.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications