[PATCH] D122079: [ADT] Add Enum matcher

Chris Bieneman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 22 20:57:47 PDT 2022


beanz added a comment.

In D122079#3401411 <https://reviews.llvm.org/D122079#3401411>, @MaskRay wrote:

> Does the template compile to efficient code? lld/ELF has a utility which seems pretty efficient https://reviews.llvm.org/D112385#3084969

With a constant value it will fold to true/false, with non-constant values it optimizes to minimal range checks.

In D122079#3401413 <https://reviews.llvm.org/D122079#3401413>, @MaskRay wrote:

> Usage link is usually directly mentioned to justify an addition to ADT/ or Support/.

This is part of a commit stack. The next commit in the stack which uses it is https://reviews.llvm.org/D122081. :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122079



More information about the llvm-commits mailing list