[PATCH] D143976: [ADT] Add lookupOrTrap method to DenseMap and StringMap

Chris Lattner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 14 20:38:31 PST 2023


lattner added a comment.

We generally don't worry to much about compatibility issues.  It is good to deprecate the old name for a few weeks before removing it, but we don't do long cycles here.

I also don't like the 'at' name particularly, but it is highly precedented by the STL.  I think any alternate name would have to be really good to overcome that.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143976/new/

https://reviews.llvm.org/D143976



More information about the llvm-commits mailing list