[all-commits] [llvm/llvm-project] 6f7737: [ImplicitNullChecks] NFC: Separated out checks and...

[annamthomas via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 6f7737c46811993c0ed9b9143cbe4cb49dcf1d03
      https://github.com/llvm/llvm-project/commit/6f7737c46811993c0ed9b9143cbe4cb49dcf1d03
  Author: Anna Thomas <anna at azul.com>
  Date:   2020-09-02 (Wed, 02 Sep 2020)

  Changed paths:
    M llvm/lib/CodeGen/ImplicitNullChecks.cpp

  Log Message:
  -----------
  [ImplicitNullChecks] NFC: Separated out checks and added comments

Separated out some checks in isSuitableMemoryOp and added comments
explaining why some of those checks are done.

Tests-Run:X86 implicit null checks tests.


  Commit: 425573a2fa2dc5666273944a584acdb286447b66
      https://github.com/llvm/llvm-project/commit/425573a2fa2dc5666273944a584acdb286447b66
  Author: Anna Thomas <anna at azul.com>
  Date:   2020-09-02 (Wed, 02 Sep 2020)

  Changed paths:
    M llvm/lib/CodeGen/ImplicitNullChecks.cpp

  Log Message:
  -----------
  [ImplicitNullChecks] NFC: Refactor dependence safety check

After computing dependence, we check if it is safe to hoist by
identifying if it clobbers any liveIns in the sibling block (NullSucc).
This check is moved to its own function which will be used in the
soon-to-be modified dependence checking algorithm for implicit null
checks pass.

Tests-Run: lit tests on X86/implicit-*


Compare: https://github.com/llvm/llvm-project/compare/255a60cdd6fd...425573a2fa2d