This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] Fix a bug in the cmakelists file when CMAKE_CXX_FLAGS are empty
ClosedPublic

Authored by stella.stamenova on Dec 3 2020, 10:17 AM.

Details

Summary

Right now, the regex expression will fail if the flags were not set. Instead, we should follow the pattern of other llvm projects and quote the expression, so that it can work even when the flags are not set.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptDec 3 2020, 10:17 AM
Herald added subscribers: Restricted Project, mgorny, dberris. · View Herald Transcript
stella.stamenova requested review of this revision.Dec 3 2020, 10:17 AM
stella.stamenova added a reviewer: phosek.
phosek accepted this revision.Dec 3 2020, 10:19 AM

LGTM, thanks!

This revision is now accepted and ready to land.Dec 3 2020, 10:19 AM
This revision was landed with ongoing or failed builds.Dec 3 2020, 10:27 AM
This revision was automatically updated to reflect the committed changes.