Comments (8)
Thanks for letting us know!
Guoguo, can you please look at this?
Dan
On Mon, Oct 5, 2015 at 11:07 AM, posophe [email protected] wrote:
Hello everyone !
RPMLint has noticed a code breaking in const-arpa-lm.cc with
-strict-alising.
I: Program is likely to break with new gcc. Try -fno-strict-aliasing.
W: kaldi strict-aliasing-punning const-arpa-lm.cc:551, 555, 580, 913Kaldi has been built with gcc 5. The concerned lines are 551, 555, 580 and
913—
Reply to this email directly or view it on GitHub
#205.
from kaldi.
I'm wondering what is the best way to fix this. What I have in mind is to
use union instead of reinterpret_cast, for example, define a union as
follows:
union Int32AndFloat {
int32 i;
float f;
};
Then we replace reinterpret_cast with this union. Any ideas? If you think
this is a good solution, I'll go ahead and apply the change.
Guoguo
On Mon, Oct 5, 2015 at 2:13 PM, Daniel Povey [email protected]
wrote:
Thanks for letting us know!
Guoguo, can you please look at this?
DanOn Mon, Oct 5, 2015 at 11:07 AM, posophe [email protected] wrote:
Hello everyone !
RPMLint has noticed a code breaking in const-arpa-lm.cc with
-strict-alising.
I: Program is likely to break with new gcc. Try -fno-strict-aliasing.
W: kaldi strict-aliasing-punning const-arpa-lm.cc:551, 555, 580, 913Kaldi has been built with gcc 5. The concerned lines are 551, 555, 580
and
913—
Reply to this email directly or view it on GitHub
#205.—
Reply to this email directly or view it on GitHub
#205 (comment).
from kaldi.
Hm.
I think it will be easier to add -fno-strict-aliasing to the
EXTRA_CXXFLAGS in lm/Makefile.
Dan
On Mon, Oct 5, 2015 at 5:36 PM, chenguoguo [email protected] wrote:
I'm wondering what is the best way to fix this. What I have in mind is to
use union instead of reinterpret_cast, for example, define a union as
follows:union Int32AndFloat {
int32 i;
float f;
};Then we replace reinterpret_cast with this union. Any ideas? If you think
this is a good solution, I'll go ahead and apply the change.Guoguo
On Mon, Oct 5, 2015 at 2:13 PM, Daniel Povey [email protected]
wrote:Thanks for letting us know!
Guoguo, can you please look at this?
DanOn Mon, Oct 5, 2015 at 11:07 AM, posophe [email protected]
wrote:Hello everyone !
RPMLint has noticed a code breaking in const-arpa-lm.cc with
-strict-alising.
I: Program is likely to break with new gcc. Try -fno-strict-aliasing.
W: kaldi strict-aliasing-punning const-arpa-lm.cc:551, 555, 580, 913Kaldi has been built with gcc 5. The concerned lines are 551, 555, 580
and
913—
Reply to this email directly or view it on GitHub
#205.—
Reply to this email directly or view it on GitHub
#205 (comment).—
Reply to this email directly or view it on GitHub
#205 (comment).
from kaldi.
Actually, after reading this intro to the strict-aliasing rule
http://cellperformance.beyond3d.com/articles/2006/06/understanding-strict-aliasing.html
I think it would be better to make it a union, as you suggested.
Dan
On Mon, Oct 5, 2015 at 7:00 PM, Daniel Povey [email protected] wrote:
Hm.
I think it will be easier to add -fno-strict-aliasing to the
EXTRA_CXXFLAGS in lm/Makefile.
DanOn Mon, Oct 5, 2015 at 5:36 PM, chenguoguo [email protected]
wrote:I'm wondering what is the best way to fix this. What I have in mind is to
use union instead of reinterpret_cast, for example, define a union as
follows:union Int32AndFloat {
int32 i;
float f;
};Then we replace reinterpret_cast with this union. Any ideas? If you think
this is a good solution, I'll go ahead and apply the change.Guoguo
On Mon, Oct 5, 2015 at 2:13 PM, Daniel Povey [email protected]
wrote:Thanks for letting us know!
Guoguo, can you please look at this?
DanOn Mon, Oct 5, 2015 at 11:07 AM, posophe [email protected]
wrote:Hello everyone !
RPMLint has noticed a code breaking in const-arpa-lm.cc with
-strict-alising.
I: Program is likely to break with new gcc. Try -fno-strict-aliasing.
W: kaldi strict-aliasing-punning const-arpa-lm.cc:551, 555, 580, 913Kaldi has been built with gcc 5. The concerned lines are 551, 555, 580
and
913—
Reply to this email directly or view it on GitHub
#205.—
Reply to this email directly or view it on GitHub
#205 (comment).—
Reply to this email directly or view it on GitHub
#205 (comment).
from kaldi.
Just sent a pull request... I'll use the union solution then..
Guoguo
On Mon, Oct 5, 2015 at 7:14 PM, Daniel Povey [email protected]
wrote:
Actually, after reading this intro to the strict-aliasing rule
http://cellperformance.beyond3d.com/articles/2006/06/understanding-strict-aliasing.html
I think it would be better to make it a union, as you suggested.
DanOn Mon, Oct 5, 2015 at 7:00 PM, Daniel Povey [email protected] wrote:
Hm.
I think it will be easier to add -fno-strict-aliasing to the
EXTRA_CXXFLAGS in lm/Makefile.
DanOn Mon, Oct 5, 2015 at 5:36 PM, chenguoguo [email protected]
wrote:I'm wondering what is the best way to fix this. What I have in mind is
to
use union instead of reinterpret_cast, for example, define a union as
follows:union Int32AndFloat {
int32 i;
float f;
};Then we replace reinterpret_cast with this union. Any ideas? If you
think
this is a good solution, I'll go ahead and apply the change.Guoguo
On Mon, Oct 5, 2015 at 2:13 PM, Daniel Povey [email protected]
wrote:Thanks for letting us know!
Guoguo, can you please look at this?
DanOn Mon, Oct 5, 2015 at 11:07 AM, posophe [email protected]
wrote:Hello everyone !
RPMLint has noticed a code breaking in const-arpa-lm.cc with
-strict-alising.
I: Program is likely to break with new gcc. Try
-fno-strict-aliasing.
W: kaldi strict-aliasing-punning const-arpa-lm.cc:551, 555, 580, 913Kaldi has been built with gcc 5. The concerned lines are 551, 555,
580
and
913—
Reply to this email directly or view it on GitHub
#205.—
Reply to this email directly or view it on GitHub
<#205 (comment)
.—
Reply to this email directly or view it on GitHub
#205 (comment).—
Reply to this email directly or view it on GitHub
#205 (comment).
from kaldi.
is this resolved?
@chenguoguo you can explicitly mention the PR # so that these two things would get crosslinked
Is it #211 ?
from kaldi.
Yeah it has been checked in.
Guoguo
On Fri, Oct 16, 2015 at 3:04 PM, jtrmal [email protected] wrote:
is this resolved?
—
Reply to this email directly or view it on GitHub
#205 (comment).
from kaldi.
Closing, as it's been resolved in #211
from kaldi.
Related Issues (20)
- Seeking Guidance on Custom Urdu ASR Training Data and Vocabulary Expansion
- KaldiFatalError HOT 1
- NO_PUBKEY on intell error HOT 1
- The `Resize` function doesn't change `stride_` if resize_type == kSetZero and `rows == MatrixBase<Real>::num_rows_ && cols == MatrixBase<Real>::num_cols_`.
- Integrate ZLUDA for AMD CUDA HOT 1
- issue with install_klm.sh HOT 2
- utils/validate_data_dir.sh: Error: in data/train_shorter, recording-ids extracted from wav.scp and reco2dur file
- ./install_mkl.sh error: HOT 4
- windows cuda 12.1 nvtools are no longer part of cuda installer HOT 4
- sh extras/check_dependencies.sh HOT 1
- Identifier "FLT_MAX" is undefined HOT 6
- Will plda pre-trained models from callhome be provided for 5fold cross-validation, which has been the mainstream approach on this dataset recently HOT 1
- [NEW RESOURCE] Introducing MARS5, open-source, insanely prosodic text-to-speech (TTS) model.
- Empty Recordings File in utils/fix_data_dir.sh Script
- Misleading description in 'fstmakecontextsyms.cc' file
- Can't build tools on Ubuntu 24.04 due to dependency on python 2.7 HOT 3
- About the implementation of toolkits to retrieve any time-aligned information typically use in research and industry HOT 4
- Can't initialise KaldiRecognizer with a simple list of words HOT 2
- Implicit fallthroughs HOT 1
- fMLLR transforms failed. report: Failed to read vector from stream. Error reading vector data (binary mode); truncated stream? (size = 38) File position at start is -1, currently -1 HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from kaldi.