[Patch] Avoid conversion to float when creating ConstantDataArray/ConstantDataVector
Rafael Espíndola
rafael.espindola at gmail.com
Thu Feb 5 13:50:21 PST 2015
On 5 February 2015 at 16:34, Raoux, Thomas F <thomas.f.raoux at intel.com> wrote:
> There is a test as part of the patch. Let me know if any other tests are needed.
Duh, I applied the patch to make it easier to read and didn't notice
the test. Sorry about that. The test only checks the vector code path,
no?
> The spec already specify that bitcast doesn't change any bits. Although here it is not the bitcast changing the bits but the insertElement. I would expect that the insertElement shouldn't change the bits either.
>
> "The ‘bitcast‘ instruction converts value to type ty2 without changing any bits."
Cool. I think the patch is fine. Nick, would you mind double checking?
Cheers,
Rafael
More information about the llvm-commits
mailing list