[PATCH] D150879: Add control of hex casing in APInt::toString

Chris Lattner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 18 10:47:53 PDT 2023


lattner added a comment.

I would suggest a silly microoptimization:

  static const char BothDigits[] = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789abcdefghijklmnopqrstuvwxyz";
  const char *Digits = BothDigits+UpperCase ? 36 :0;

Avoids looking up two different GOT entries.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150879



More information about the llvm-commits mailing list