[all-commits] [llvm/llvm-project] ee414e: [CI] Refactor out some early exits in compute_proj...

Aiden Grossman via All-commits all-commits at lists.llvm.org
Sun Jun 22 08:08:50 PDT 2025


  Branch: refs/heads/users/boomanaiden154/ci-use-github-native-groups-in-monolithic-scripts
  Home:   https://github.com/llvm/llvm-project
  Commit: ee414e35047c588daba1ee62e8291ea171700f43
      https://github.com/llvm/llvm-project/commit/ee414e35047c588daba1ee62e8291ea171700f43
  Author: Aiden Grossman <aidengrossman at google.com>
  Date:   2025-06-22 (Sun, 22 Jun 2025)

  Changed paths:
    M .ci/compute_projects.py

  Log Message:
  -----------
  [CI] Refactor out some early exits in compute_projects

I have a habit of using early exits given it is in the LLVM coding
standards, but most of the early exits used within this script were
trivial and actually adding complexity. These are all instances where we
only perform one operation after the early exit, so removing the early
exit means less lines of code and arguably more readable code.

Reviewers: DavidSpickett, tstellar, cmtice, lnihlen

Reviewed By: DavidSpickett

Pull Request: https://github.com/llvm/llvm-project/pull/143478


  Commit: f18682a9c3799f9040660fbc7609b49ffe464bb5
      https://github.com/llvm/llvm-project/commit/f18682a9c3799f9040660fbc7609b49ffe464bb5
  Author: Aiden Grossman <agrossman154 at yahoo.com>
  Date:   2025-06-22 (Sun, 22 Jun 2025)

  Changed paths:

  Log Message:
  -----------
  [𝘀𝗽𝗿] changes introduced through rebase

Created using spr 1.3.6

[skip ci]


  Commit: 780ca57430435dbfef1b339fead1cb0351e01a75
      https://github.com/llvm/llvm-project/commit/780ca57430435dbfef1b339fead1cb0351e01a75
  Author: Aiden Grossman <agrossman154 at yahoo.com>
  Date:   2025-06-22 (Sun, 22 Jun 2025)

  Changed paths:

  Log Message:
  -----------
  rebase

Created using spr 1.3.6


Compare: https://github.com/llvm/llvm-project/compare/d106caab42d3...780ca5743043

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list