[PATCH] D37989: InstCombine: Insert missing canonicalizes

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 18 12:08:42 PDT 2017


arsenm created this revision.
Herald added subscribers: nhaehnle, wdng.

minnum/maxnum are supposed to return a canonicalized value.
We were folding cases where one of the operands is constant
without inserting the appropriate canonicalize. Also fix
the LangRef to mention the canonicalize part of the IEEE
definition.


https://reviews.llvm.org/D37989

Files:
  docs/LangRef.rst
  include/llvm/IR/IRBuilder.h
  lib/IR/IRBuilder.cpp
  lib/Transforms/InstCombine/InstCombineCalls.cpp
  test/Transforms/InstCombine/AMDGPU/amdgcn-intrinsics.ll
  test/Transforms/InstCombine/maxnum.ll
  test/Transforms/InstCombine/minnum.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37989.115698.patch
Type: text/x-patch
Size: 12002 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170918/e064d7e1/attachment.bin>


More information about the llvm-commits mailing list