[PATCH] D30834: [x86] these aren't the undefs you're looking for (PR32176)
Craig Topper via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 10 15:42:55 PST 2017
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
More information about the cfe-commits
mailing list