Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 63868edc authored by George Burgess IV's avatar George Burgess IV Committed by android-build-merger
Browse files

Merge "Disable dtor inlining for clang-tidy" am: e51fcf01

am: e19014c9

Change-Id: Id9ab3c326c1ba5ade3398beadd5b1a87139ff7a4
parents 6516a40a e19014c9
Loading
Loading
Loading
Loading
+11 −2
Original line number Diff line number Diff line
@@ -1724,10 +1724,19 @@ ifneq (,$(filter 1 true,$(my_tidy_enabled)))
      my_tidy_flags += -quiet -extra-arg-before=-fno-caret-diagnostics
    endif

    ifneq ($(my_tidy_checks),)
      # We might be using the static analyzer through clang-tidy.
      # https://bugs.llvm.org/show_bug.cgi?id=32914
    ifneq ($(my_tidy_checks),)
      my_tidy_flags += -extra-arg-before=-D__clang_analyzer__

      # A recent change in clang-tidy (r328258) enabled destructor inlining,
      # which appears to cause a number of false positives. Until that's
      # resolved, this turns off the effects of r328258.
      # https://bugs.llvm.org/show_bug.cgi?id=37459
      my_tidy_flags += -extra-arg-before=-Xclang
      my_tidy_flags += -extra-arg-before=-analyzer-config
      my_tidy_flags += -extra-arg-before=-Xclang
      my_tidy_flags += -extra-arg-before=c++-temp-dtor-inlining=false
    endif
  endif
endif