main: stop running the LLVM verifier for trivial changes

The above changes might indeed introduce inconsistencies in the IR, but
the code is small and doesn't change often so it's unnecessary to always
check for errors. It will be tested again later anyway.

The compile time impact was somewhere around 6%, so that's a nice
improvement.
This commit is contained in:
Ayke van Laethem
2019-09-16 13:57:40 +02:00
committed by Ron Evans
parent 4eb34b36f8
commit b4859240e1
-6
View File
@@ -154,9 +154,6 @@ func Compile(pkgName, outpath string, spec *TargetSpec, config *BuildConfig, act
if spec.GOOS != "darwin" {
c.ApplyFunctionSections() // -ffunction-sections
}
if err := c.Verify(); err != nil {
return errors.New("verification error after applying function sections")
}
// Browsers cannot handle external functions that have type i64 because it
// cannot be represented exactly in JavaScript (JS only has doubles). To
@@ -168,9 +165,6 @@ func Compile(pkgName, outpath string, spec *TargetSpec, config *BuildConfig, act
if err != nil {
return err
}
if err := c.Verify(); err != nil {
return errors.New("verification error after running the wasm i64 hack")
}
}
// Optimization levels here are roughly the same as Clang, but probably not