Remove -U_FORTIFY_SOURCE when thin_lto is enabled#15433
Merged
ckolli5 merged 1 commit intobazelbuild:release-5.2.0from May 9, 2022
ckolli5:ck/remove-ufortify-source
Merged
Remove -U_FORTIFY_SOURCE when thin_lto is enabled#15433ckolli5 merged 1 commit intobazelbuild:release-5.2.0from ckolli5:ck/remove-ufortify-source
ckolli5 merged 1 commit intobazelbuild:release-5.2.0from
ckolli5:ck/remove-ufortify-source
Conversation
Loading
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Clang does not accept
-Uflags when doing lto backend actions:This isn't a problem for
-Dflags because of a special case:https://github.com/llvm/llvm-project/blob/10e99eb7e4adbb8e2407e2f6ae3d2e6420d060c9/clang/lib/Driver/ToolChains/Clang.cpp#L5579-L5582
Closes #15007.
PiperOrigin-RevId: 440314037