Skip to content

Commit

Permalink
[MLGO] Remove absl dep from libraries
Browse files Browse the repository at this point in the history
The library files in the new mlgo-utils utilities folder only depend on
absl.logging. The builtin Python logging library is a direct drop-in
replacement here, so we can just change the include and drop the test
dependency.
  • Loading branch information
boomanaiden154 committed Jan 21, 2024
1 parent bb6564a commit 39f1ca5
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
import os
import json
import glob

from absl import logging
import logging

_FILE_NAME = "corpus_description.json"

Expand Down
3 changes: 1 addition & 2 deletions llvm/utils/mlgo-utils/mlgo/corpus/extract_ir_lib.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,11 +11,10 @@
import multiprocessing
import functools
import json
import logging

from typing import Dict, List, Optional

from absl import logging

_UNSPECIFIED_OVERRIDE = ["<UNSPECIFIED>"]


Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# REQUIRES: python-38, absl
# REQUIRES: python-38

## Test the functionality of combine_training_corpus_lib

Expand Down
2 changes: 1 addition & 1 deletion llvm/utils/mlgo-utils/tests/corpus/extract_ir_test.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# REQUIRES: python-38, absl
# REQUIRES: python-38

## Test the functionality of extract_ir_lib

Expand Down
2 changes: 1 addition & 1 deletion llvm/utils/mlgo-utils/tests/corpus/make_corpus_test.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# REQUIRES: python-38, absl
# REQUIRES: python-38

## Test the functionality of make_corpus_lib

Expand Down

0 comments on commit 39f1ca5

Please sign in to comment.