Skip to content

Commit

Permalink
remove pcre library, use ocaml str instead
Browse files Browse the repository at this point in the history
PCRE relies on a shared library, precluding its use in Javascript.
instead, we use the standard ocaml "str" library. note, however,
that this supports a smaller subscript of regex (notably no negative
lookahead), and has a different syntax.

https://ocaml.org/manual/5.2/api/Str.html
  • Loading branch information
katrinafyi committed Jun 20, 2024
1 parent 44de990 commit 75cf86f
Show file tree
Hide file tree
Showing 7 changed files with 11 additions and 11 deletions.
1 change: 0 additions & 1 deletion asli.opam
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ depends: [
"pprint"
"zarith"
"z3" {>= "4.8.7"}
"pcre"
"alcotest" {with-test}
"dune-site"
"lwt"
Expand Down
4 changes: 2 additions & 2 deletions bin/dune
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
(modes exe byte)
(modules asli)
(flags (-cclib -lstdc++))
(libraries libASL linenoise pprint pcre)
(libraries libASL linenoise pprint)
)

(executable
Expand All @@ -14,7 +14,7 @@
(modes exe)
(modules server)
(flags (-cclib -lstdc++))
(libraries libASL pprint pcre lwt.unix yojson cohttp-lwt cohttp-lwt-unix))
(libraries libASL pprint lwt.unix yojson cohttp-lwt cohttp-lwt-unix))


(executable
Expand Down
1 change: 0 additions & 1 deletion dune-project
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@
"pprint"
"zarith"
("z3" (>= "4.8.7"))
"pcre"
("alcotest" :with-test)
"dune-site"
"lwt"
Expand Down
4 changes: 2 additions & 2 deletions libASL/decoder_program.ml
Original file line number Diff line number Diff line change
Expand Up @@ -116,10 +116,10 @@ let run iset pat env problematic =

(* Find all matching instructions, pulled from testing.ml *)
let decoder = Eval.Env.getDecoder env (Ident iset) in
let re = Pcre.regexp pat in
let re = Str.regexp pat in
let filterfn = function
| ((Encoding_Block (Ident nm, Ident is, _, _, _, _, _, _)),_,_,_) ->
is = iset && Pcre.pmatch ~rex:re nm && not (List.mem nm problematic)
is = iset && Str.string_match re nm 0 && not (List.mem nm problematic)
| _ -> assert false
in
let encs = List.filter filterfn (Eval.Env.listInstructions env) in
Expand Down
2 changes: 1 addition & 1 deletion libASL/dune
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
offline_transform ocaml_backend dis_tc
offline_opt
)
(libraries pprint zarith z3 str pcre dune-site))
(libraries pprint zarith z3 str dune-site))

(generate_sites_module
(module res)
Expand Down
4 changes: 2 additions & 2 deletions libASL/testing.ml
Original file line number Diff line number Diff line change
Expand Up @@ -423,10 +423,10 @@ let op_test_opcode (env: Env.t) (iset: string) (op: int): Env.t opresult =
let get_opcodes (opt_verbose: bool ref) (iset: string) (instr: string) (env: Env.t): (string * instr_field list * ((int * bool) list) option) list =
if !opt_verbose then Printf.printf "Coverage for encoding %s\n" instr;

let re = Pcre.regexp instr in
let re = Str.regexp instr in
let encoding_matches = function
| (Encoding_Block (Ident nm, Ident is, _, _, _, _, _, _)) ->
is = iset && Pcre.pmatch ~rex:re nm
is = iset && Str.string_match re nm 0
| _ -> assert false
in
let encs = List.map (fun (x,_,_,_) -> x) (Env.listInstructions env) in
Expand Down
6 changes: 4 additions & 2 deletions tests/coverage/dune
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,10 @@
(rule (with-stdout-to cov_branch (run ./run "aarch64_branch.+")))
(rule (with-stdout-to cov_float (run ./run "aarch64_float_.+")))
(rule (with-stdout-to cov_vector1 (run ./run "aarch64_vector_arithmetic_binary.+")))
(rule (with-stdout-to cov_vector2 (run ./run "aarch64_vector_(?!arithmetic_binary).+")))
(rule (with-stdout-to cov_vector2 (run ./run "aarch64_vector_\\(arithmetic_unary\\|[^a]\\).+")))
(rule (with-stdout-to cov_memory (run ./run "aarch64_memory_.+")))
; note: cov_vector2 is intended to be every vector instruction that's not arithmetic_binary.
; note ocaml Str regex syntax: https://ocaml.org/manual/5.2/api/Str.html

(rule (alias coverage) (action (diff ./aarch64_integer cov_integer)))
(rule (alias coverage) (action (diff ./aarch64_branch cov_branch)))
Expand All @@ -34,7 +36,7 @@
(rule (with-stdout-to off_cov_branch (run ./off_run "aarch64_branch.+")))
(rule (with-stdout-to off_cov_float (run ./off_run "aarch64_float_.+")))
(rule (with-stdout-to off_cov_vector1 (run ./off_run "aarch64_vector_arithmetic_binary.+")))
(rule (with-stdout-to off_cov_vector2 (run ./off_run "aarch64_vector_(?!arithmetic_binary).+")))
(rule (with-stdout-to off_cov_vector2 (run ./off_run "aarch64_vector_\\(arithmetic_unary\\|[^a]\\).+")))
(rule (with-stdout-to off_cov_memory (run ./off_run "aarch64_memory_.+")))

(rule (alias offline-coverage) (action (diff ./aarch64_integer off_cov_integer)))
Expand Down

0 comments on commit 75cf86f

Please sign in to comment.