[PATCH] D17219: [x86-64] allow mfence even with -mno-sse (PR23203)

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 12 15:09:18 PST 2016


spatel created this revision.
spatel added reviewers: jfb, craig.topper, majnemer.
spatel added a subscriber: llvm-commits.
Herald added a subscriber: mcrosier.

As shown in:
https://llvm.org/bugs/show_bug.cgi?id=23203
...we currently die because lowering believes that mfence is allowed without SSE2 on x86-64, but the instruction def doesn't know that.
I don't know if this is right, but if not, at least it's consistently wrong. :)

http://reviews.llvm.org/D17219

Files:
  lib/Target/X86/X86ISelLowering.cpp
  lib/Target/X86/X86InstrInfo.td
  lib/Target/X86/X86InstrSSE.td
  lib/Target/X86/X86Subtarget.h
  test/CodeGen/X86/mfence.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17219.47861.patch
Type: text/x-patch
Size: 4526 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160212/a8792271/attachment.bin>


More information about the llvm-commits mailing list