[PATCH] D51215: Fix misfolding of IRBuilder.CreateICmp(int_ty X, bitcast (float_ty Y) to int_ty)
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 12 13:54:11 PST 2019
spatel added a comment.
Herald added a subscriber: jdoerfert.
This took some work, but I have an IR test case that should show the bug:
I haven't looked at D51216 <https://reviews.llvm.org/D51216> yet, so this might be a dumb question, but is there an fcmp sibling to this bug?
Comment at: unittests/IR/IRBuilderTest.cpp:679
+ // fp->int bitcast.
+ auto F32TimesTwo = Builder.CreateAdd(F32, F32);
+ auto I32 = Builder.CreateBitCast(F32TimesTwo, I32Type);
This means that IRBuilder has a bug/opportunity to be stricter? How did we create an integer add with FP operands?
CHANGES SINCE LAST ACTION
More information about the llvm-commits