You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This ticket tracks getting an LTO example working in node-cpp-skel that other modules can follow. This is worthwhile but a little hairy since it requires:
the same compiler to be used for compiling all binaries (tricky with mason unless you ask mason to source compile)
it requires upgrading binutils from mason to use the "gold" linker within binutils for linux
Ideally as part of this effort we also document an exact, measurable situation when LTO actually helps.
Never did this. Still worthwhile. The reason I merged without doing a perf validation benchmark is that:
I noticed that adding -flto can catch ODR errors that are otherwise silent. So making sure we can compile with LTO increases the robustness of the code
I wanted to support fsanitize=cfi which requires it
Activity
always enable LTO in release mode - refs #110
springmeyer commentedon Apr 24, 2018
now enabled after 93edf4f
springmeyer commentedon Aug 9, 2018
Never did this. Still worthwhile. The reason I merged without doing a perf validation benchmark is that:
fsanitize=cfiwhich requires it