[all-commits] [llvm/llvm-project] e2b75c: [NFCI][InstCombine] Move store merging from `visit...

Roman Lebedev via All-commits all-commits at lists.llvm.org
Tue Jul 14 00:42:51 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: e2b75cafcbacd0e4296fed65afca7197aef172aa
      https://github.com/llvm/llvm-project/commit/e2b75cafcbacd0e4296fed65afca7197aef172aa
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2020-07-14 (Tue, 14 Jul 2020)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstCombineInternal.h
    M llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp
    M llvm/lib/Transforms/InstCombine/InstructionCombining.cpp

  Log Message:
  -----------
  [NFCI][InstCombine] Move store merging from `visitStoreInst()` into `visitUnconditionalBranchInst()`

Summary:
As @nikic is pointing out in https://bugs.llvm.org/show_bug.cgi?id=46680#c5,
InstCombine should not have forward instruction scans,
so let's move this transform into the proper place.

This is pretty much NFCI.

Reviewers: nikic, spatel

Reviewed By: nikic

Subscribers: hiraditya, llvm-commits, nikic

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D83670




More information about the All-commits mailing list