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
After #1732 , the CMAKE_INTERPROCEDURAL_OPTIMIZATION global CMake option is always enabled if the compiler supports it. In gcc, this adds the -flto flag, that cannot be removed via UA_ARCH_REMOVE_FLAGS because it is added by CMake itself.
However, in some cases you may want to disable IPO even if your compilers supports it, for example if your linker has problem with it. In those cases, it would be convenient to have a UA_ENABLE_INTERPROCEDURAL_OPTIMIZATION CMake option, that by default is ON and that would permit to disable the IPO.
If you are ok with this idea, I would be happy to open a PR implementing it.
Checklist
Please provide the following information:
open62541 Version (release number or git tag): v1.0
Other OPC UA SDKs used (client or server):
Operating system:
Logs (with UA_LOGLEVEL set as low as necessary) attached
Wireshark network dump attached
Self-contained code example attached
Critical issue: no
The text was updated successfully, but these errors were encountered:
traversaro
changed the title
Add option to disable
Add option to disable interprocedural optimization
Oct 25, 2019
traversaro
changed the title
Add option to disable interprocedural optimization
Add UA_ENABLE_INTERPROCEDURAL_OPTIMIZATION CMake option to disable interprocedural optimization
Oct 25, 2019
Description
After #1732 , the
CMAKE_INTERPROCEDURAL_OPTIMIZATION
global CMake option is always enabled if the compiler supports it. In gcc, this adds the-flto
flag, that cannot be removed viaUA_ARCH_REMOVE_FLAGS
because it is added by CMake itself.However, in some cases you may want to disable IPO even if your compilers supports it, for example if your linker has problem with it. In those cases, it would be convenient to have a
UA_ENABLE_INTERPROCEDURAL_OPTIMIZATION
CMake option, that by default isON
and that would permit to disable the IPO.If you are ok with this idea, I would be happy to open a PR implementing it.
Checklist
Please provide the following information:
UA_LOGLEVEL
set as low as necessary) attachedThe text was updated successfully, but these errors were encountered: