Skip to content

Commit

Permalink
moving options -Wall before extra_cxx_flags to resolve conflicting -W…
Browse files Browse the repository at this point in the history
…no-XXX options for clang (#945)
  • Loading branch information
troy4eg authored Dec 6, 2023
1 parent d685b6c commit 0a1b9f0
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions compiler/compiler-settings.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -284,11 +284,13 @@ void CompilerSettings::init() {

remove_extra_spaces(extra_cxx_flags.value_);
std::stringstream ss;
ss << extra_cxx_flags.get();
ss << " -iquote" << kphp_src_path.get()
<< " -iquote " << kphp_src_path.get() << "objs/generated/auto/runtime";
ss << " -Wall -fwrapv -Wno-parentheses -Wno-trigraphs";
ss << " -fno-strict-aliasing -fno-omit-frame-pointer";
ss << "-Wall "
<< extra_cxx_flags.get()
<< " -iquote" << kphp_src_path.get()
<< " -iquote " << kphp_src_path.get()
<< "objs/generated/auto/runtime"
<< " -fwrapv -Wno-parentheses -Wno-trigraphs"
<< " -fno-strict-aliasing -fno-omit-frame-pointer";
#ifdef __x86_64__
ss << " -march=sandybridge";
#elif __aarch64__
Expand Down

0 comments on commit 0a1b9f0

Please sign in to comment.