Skip to content

Commit

Permalink
Add a bit more optimization
Browse files Browse the repository at this point in the history
  • Loading branch information
nshmyrev committed Jan 9, 2021
1 parent 1b8332a commit a8ae602
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions travis/Dockerfile.manylinux
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ RUN cd /opt \
&& cd /opt/kaldi/tools \
&& git clone -b v0.3.13 --single-branch https://github.com/xianyi/OpenBLAS \
&& git clone -b v3.2.1 --single-branch https://github.com/alphacep/clapack \
&& make -C OpenBLAS ONLY_CBLAS=1 DYNAMIC_ARCH=1 TARGET=CORE2 USE_LOCKING=1 USE_THREAD=0 all \
&& make -C OpenBLAS ONLY_CBLAS=1 DYNAMIC_ARCH=1 TARGET=NEHALEM USE_LOCKING=1 USE_THREAD=0 all \
&& make -C OpenBLAS PREFIX=$(pwd)/OpenBLAS/install install \
&& mkdir -p clapack/BUILD && cd clapack/BUILD && cmake .. && make -j 10 && find . -name "*.a" | xargs cp -t ../../OpenBLAS/install/lib \
&& cd /opt/kaldi/tools \
Expand All @@ -38,5 +38,7 @@ RUN cd /opt \
&& make -j 10 && make install \
&& cd /opt/kaldi/src \
&& ./configure --mathlib=OPENBLAS_CLAPACK --shared --use-cuda=no \
&& make -j 10 online2 lm rnnlm \
&& sed -i 's:-msse -msse2:-msse -msse2:g' kaldi.mk \
&& sed -i 's: -O1 : -O3 :g' kaldi.mk \
&& make -j $(nproc) online2 lm rnnlm \
&& find /opt/kaldi -name "*.o" -exec rm {} \;

3 comments on commit a8ae602

@svenha
Copy link

@svenha svenha commented on a8ae602 Jan 9, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s:-msse -msse2:-msse -msse2:g looks strange.

BTW: you can combine several sed replacements as follows: sed -i -e RE1 -e RE2

@nshmyrev
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s:-msse -msse2:-msse -msse2:g looks strange.

BTW: you can combine several sed replacements as follows: sed -i -e RE1 -e RE2

This is a placeholder in place I want to put sse4 there in the future

@svenha
Copy link

@svenha svenha commented on a8ae602 Jan 9, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tricky. Thanks for explaining.

Please sign in to comment.