[PATCH] D45572: [X86] Replace action Promote with Custom for operation ISD::SINT_TO_FP

Vyacheslav via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 16 07:43:35 PDT 2018


vrybalov added inline comments.


================
Comment at: test/CodeGen/X86/sitofp.ll:3
+
+target triple = "i386-unknown-linux-gnu"
+
----------------
RKSimon wrote:
> This test needs improving - you need to add proper FileCheck testing, preferably use the update_llc_test_checks script to check all codegen and cleanup the actual test - I'm not convinced you need all this code to demonstrate the sitofp?
Main purpose of the test is to check llc doesn't hang on compilation. Now it hangs.


Repository:
  rL LLVM

https://reviews.llvm.org/D45572





More information about the llvm-commits mailing list