serge-sans-paille added a comment.

In D140800#4021228 <https://reviews.llvm.org/D140800#4021228>, @thakis wrote:

> Could we make this an optional opt-in maybe? I imagine for the majority of 
> the tools, this has no measurable effect, but it currently makes the Opt 
> library harder to use for every client.

Sure, let's have two constructors, one that uses a precomputed table and one 
that does the precomputation.
Would you prefer having only clang use the faster-but-less efficient method, or 
can I keep the changes as is?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140800

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to