Comment # 28
on bug 10798
from Guy Harris
Some poking at the clang SVN repository found the revision number 144073:
efriedma | 2011-11-07 20:13:51 -0800 (Mon, 07 Nov 2011)
Misc fixes to pcmp*stri.
which
1) changed the definitions of _mm_cmpistr[acosz]() in
lib/Headers/smmintrin.h to take 3 rather than 5 arguments and to pass those 3,
rather than 5, arguments to __builtin_ia32_pcmpistri[acosz]128()
and
2) comments out, with the comment
// FIXME: These builtins are horribly broken; reenable when PR11305 is fixed.
lines in include/clang/Basic/BuiltinsX86.def that enable the
__builtin_ia32_pcmp[ie]stri[acosz]128() builtins in the first place!
This might produce the broken clangs I'm seeing on my Lion machine.
So it might be that:
compilers before that revision generate bad code;
compilers after that revision but before 161318 and 161319 may give compile
errors if you try to use those builtins;
compilers after those revisions may work OK.
Those two are
ctopper | 2012-08-05 23:22:36 -0700 (Sun, 05 Aug 2012)
Implement proper handling for pcmpistri/pcmpestri intrinsics. Requires
custom handling in DAGISelToDAG due to limitations in TableGen's implicit def
handling. Fixes PR11305.
and
ctopper | 2012-08-06 00:07:06 -0700 (Mon, 06 Aug 2012)
Re-enable pcmpistri/pcmpestri builtins in clang now that llvm supports them
properly.
You are receiving this mail because:
- You are watching all bug changes.