[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