[all-commits] [llvm/llvm-project] f42136: ValueTracking: Check instruction is in a parent in...

Matt Arsenault via All-commits all-commits at lists.llvm.org
Thu May 18 04:22:20 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f42136d4d6e102659c6a4a870349aae6425a3ba9
      https://github.com/llvm/llvm-project/commit/f42136d4d6e102659c6a4a870349aae6425a3ba9
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2023-05-18 (Thu, 18 May 2023)

  Changed paths:
    M llvm/lib/Analysis/ValueTracking.cpp
    M llvm/lib/Transforms/Utils/CloneFunction.cpp
    A llvm/test/Transforms/Inline/simplify-instruction-computeKnownFPClass-context.ll

  Log Message:
  -----------
  ValueTracking: Check instruction is in a parent in computeKnownFPClass

For some reason the inliner calls simplifyInstruction with disembodied
instructions. I consider this to be an API defect. Either the instruction
should always be inserted prior to simplification, or we at least
should pass in the new function for the context.




More information about the All-commits mailing list