[all-commits] [llvm/llvm-project] cdceef: [Analysis] Ensure we include CommandLine.h if we d...

Simon Pilgrim via All-commits all-commits at lists.llvm.org
Tue Jun 23 04:31:43 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: cdceef4a4f9cecf76c2c1f82bbdd1747fbfb75dc
      https://github.com/llvm/llvm-project/commit/cdceef4a4f9cecf76c2c1f82bbdd1747fbfb75dc
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2020-06-23 (Tue, 23 Jun 2020)

  Changed paths:
    M llvm/lib/Analysis/CaptureTracking.cpp
    M llvm/lib/Analysis/InlineAdvisor.cpp

  Log Message:
  -----------
  [Analysis] Ensure we include CommandLine.h if we declare any cl::opt flags. NFC.


  Commit: 2013be483687583d63fc7daef3347eb9906b5584
      https://github.com/llvm/llvm-project/commit/2013be483687583d63fc7daef3347eb9906b5584
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2020-06-23 (Tue, 23 Jun 2020)

  Changed paths:
    M llvm/unittests/IR/ModuleTest.cpp

  Log Message:
  -----------
  ModuleTest.cpp - fix implicit SourceMgr.h dependency. NFC.

ModuleTest.cpp was depending on other headers to define SMDiagnostic.


  Commit: e051187c540b479063911b581cbb5fdff5119a67
      https://github.com/llvm/llvm-project/commit/e051187c540b479063911b581cbb5fdff5119a67
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2020-06-23 (Tue, 23 Jun 2020)

  Changed paths:
    M llvm/tools/llvm-dwp/llvm-dwp.cpp

  Log Message:
  -----------
  llvm-dwp.cpp - fix implicit CommandLine.h dependency. NFC.

llvm-dwp uses cl::opt + cl::list but don't include CommandLine.h.


  Commit: 36bab8484d6e681a6eb1522343b273e113503ffb
      https://github.com/llvm/llvm-project/commit/36bab8484d6e681a6eb1522343b273e113503ffb
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2020-06-23 (Tue, 23 Jun 2020)

  Changed paths:
    M llvm/tools/llvm-isel-fuzzer/llvm-isel-fuzzer.cpp

  Log Message:
  -----------
  llvm-isel-fuzzer.cpp - fix implicit CommandLine.h dependency. NFC.

llvm-isel-fuzzer uses cl::opt but don't include CommandLine.h.


Compare: https://github.com/llvm/llvm-project/compare/4ffe2b24f5c7...36bab8484d6e


More information about the All-commits mailing list