Merge "Disabled avx avx2 avx512 compiler flags by default. Library components can enable the flags based on performance impact"

This commit is contained in:
Treehugger Robot 2019-06-28 21:14:33 +00:00 committed by Gerrit Code Review
commit dd5cf7574b
2 changed files with 18 additions and 5 deletions

View file

@ -67,10 +67,16 @@ var (
"sse4": []string{"-msse4"},
"sse4_1": []string{"-msse4.1"},
"sse4_2": []string{"-msse4.2"},
// Not all cases there is performance gain by enabling -mavx -mavx2
// flags so these flags are not enabled by default.
// if there is performance gain in individual library components,
// the compiler flags can be set in corresponding bp files.
// "avx": []string{"-mavx"},
// "avx2": []string{"-mavx2"},
// "avx512": []string{"-mavx512"}
"popcnt": []string{"-mpopcnt"},
"avx": []string{"-mavx"},
"avx2": []string{"-mavx2"},
"avx512": []string{"-mavx512"},
"aes_ni": []string{"-maes"},
}
)

View file

@ -86,8 +86,15 @@ var (
"sse4": []string{"-msse4"},
"sse4_1": []string{"-msse4.1"},
"sse4_2": []string{"-msse4.2"},
"avx": []string{"-mavx"},
"avx2": []string{"-mavx2"},
// Not all cases there is performance gain by enabling -mavx -mavx2
// flags so these flags are not enabled by default.
// if there is performance gain in individual library components,
// the compiler flags can be set in corresponding bp files.
// "avx": []string{"-mavx"},
// "avx2": []string{"-mavx2"},
// "avx512": []string{"-mavx512"}
"aes_ni": []string{"-maes"},
}
)