craig.topper added a comment.

Have you ran the tests all the way through to assembly and made sure we don't 
regress? If we do regress, I wouldn't hold up fixing this, but we should at 
least have bugs for what breaks.


================
Comment at: lib/CodeGen/CGBuiltin.cpp:7384
   case X86::BI__builtin_ia32_undef512:
-    return UndefValue::get(ConvertType(E->getType()));
+    // The x86 definition of "undef" is not the same as the LLVM definition
+    // (PR32176). We leave the exercise of optimizing away an unnecessary zero
----------------
Do we want to leave a TODO here to maybe do something different when we have 
freeze?


https://reviews.llvm.org/D30834



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to