[PATCH] D91600: [llvm] Cleanup llvm-yaml-numeric-parser-fuzzer

[Kirill Bobyrev via Phabricator via llvm-commits]

kbobyrev created this revision.
kbobyrev added a reviewer: kadircet.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
kbobyrev requested review of this revision.

- Use static variables instead of non-trivially destructible global ones.
- Remove unused header.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D91600

Files:
  llvm/tools/llvm-yaml-numeric-parser-fuzzer/yaml-numeric-parser-fuzzer.cpp


Index: llvm/tools/llvm-yaml-numeric-parser-fuzzer/yaml-numeric-parser-fuzzer.cpp
===================================================================
--- llvm/tools/llvm-yaml-numeric-parser-fuzzer/yaml-numeric-parser-fuzzer.cpp
+++ llvm/tools/llvm-yaml-numeric-parser-fuzzer/yaml-numeric-parser-fuzzer.cpp
@@ -9,15 +9,14 @@
 #include "llvm/ADT/StringRef.h"
 #include "llvm/Support/Regex.h"
 #include "llvm/Support/YAMLTraits.h"
-#include <cassert>
 #include <string>
 
-llvm::Regex Infinity("^[-+]?(\\.inf|\\.Inf|\\.INF)$");
-llvm::Regex Base8("^0o[0-7]+$");
-llvm::Regex Base16("^0x[0-9a-fA-F]+$");
-llvm::Regex Float("^[-+]?(\\.[0-9]+|[0-9]+(\\.[0-9]*)?)([eE][-+]?[0-9]+)?$");
-
 inline bool isNumericRegex(llvm::StringRef S) {
+  static llvm::Regex Infinity("^[-+]?(\\.inf|\\.Inf|\\.INF)$");
+  static llvm::Regex Base8("^0o[0-7]+$");
+  static llvm::Regex Base16("^0x[0-9a-fA-F]+$");
+  static llvm::Regex Float(
+      "^[-+]?(\\.[0-9]+|[0-9]+(\\.[0-9]*)?)([eE][-+]?[0-9]+)?$");
 
   if (S.equals(".nan") || S.equals(".NaN") || S.equals(".NAN"))
     return true;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91600.305696.patch
Type: text/x-patch
Size: 1068 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201117/5e4f6d31/attachment.bin>