[all-commits] [llvm/llvm-project] 9a05a8: [speculative-execution] Hoists debug values unnece...

[Carlos Alberto Enciso via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9a05a89d1ef73de7ab787071931f449935d841a7
      https://github.com/llvm/llvm-project/commit/9a05a89d1ef73de7ab787071931f449935d841a7
  Author: Carlos Alberto Enciso <carlos.alberto.enciso at gmail.com>
  Date:   2024-04-02 (Tue, 02 Apr 2024)

  Changed paths:
    M llvm/lib/Transforms/Scalar/SpeculativeExecution.cpp
    M llvm/test/Transforms/SpeculativeExecution/PR46267.ll

  Log Message:
  -----------
  [speculative-execution] Hoists debug values unnecessarily. (#85782)

After https://reviews.llvm.org/D81730:
`SpeculativeExecutionPass::considerHoistingFromTo` hoists instructions,
including debug intrinsics, as long as none of their used values are
instructions that appear prior in the block that are not being hoisted.

This behaviour has been duplicated for DPValues to get rid of a binary
difference.

The correct solution is not hoist these debug values at all, whichever
format they're in.



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