[all-commits] [llvm/llvm-project] 460469: Revert "[JumpThreading] Ignore free instructions"

Hans via All-commits all-commits at lists.llvm.org
Fri Sep 24 07:25:38 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4604695d7c20e72b551a1a5224f3de877cb41bd3
      https://github.com/llvm/llvm-project/commit/4604695d7c20e72b551a1a5224f3de877cb41bd3
  Author: Hans Wennborg <hans at chromium.org>
  Date:   2021-09-24 (Fri, 24 Sep 2021)

  Changed paths:
    M llvm/include/llvm/Transforms/Scalar/JumpThreading.h
    M llvm/lib/Transforms/Scalar/JumpThreading.cpp
    M llvm/test/Transforms/JumpThreading/free_instructions.ll
    M llvm/test/Transforms/PhaseOrdering/inlining-alignment-assumptions.ll

  Log Message:
  -----------
  Revert "[JumpThreading] Ignore free instructions"

It caused compiler crashes, see comment on the code review for repro.

> This is basically D108837 but for jump threading. Free instructions
> should be ignored for the threading decision. JumpThreading already
> skips some free instructions (like pointer bitcasts), but does not
> skip various free intrinsics -- in fact, it currently gives them a
> fairly large cost of 2.
>
> Differential Revision: https://reviews.llvm.org/D110290

This reverts commit 1e3c6fc7cb9d2ee6a5328881f95d6643afeadbff.




More information about the All-commits mailing list