[PATCH] D51215: Fix misfolding of IRBuilder.CreateICmp(int_ty X, bitcast (float_ty Y) to int_ty)

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:
rL353883 <https://reviews.llvm.org/rL353883>

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?

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D51215/new/

https://reviews.llvm.org/D51215