[PATCH] D102707: Fix non-global-value-max-name-size not considered by LLParser

[Hasyimi Bahrudin via Phabricator via llvm-commits]

hasyimibhar added inline comments.


================
Comment at: llvm/include/llvm/IR/ValueSymbolTable.h:136
   ValueMap vmap;                    ///< The map that holds the symbol table.
+  int maxNameSize; ///< The maximum size for each name. If the limit is
+                   ///< exceeded, the name is capped.
----------------
`maxNameSize` is an `int` instead of `unsigned` because there is a test (`llvm/test/Bitcode/value-with-long-name.ll`) which checks for the case where `-non-global-value-max-name-size=0`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102707