[PATCH] D99362: Verify the LLVMContext for Attributes.

Nick Lewycky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 2 14:48:38 PDT 2021


nickwasmer added a comment.

> Other than it not being "const" is there any reason you couldn't use FindNodeOrInsertPos to implement contains?

That works, I don't recall why I thought it didn't. With that, I can remove the changes to FoldingSet entirely. Thank you!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99362



More information about the llvm-commits mailing list