This revision was automatically updated to reflect the committed changes.
Closed by commit rL331249: [X86] directstore and movdir64b intrinsics (authored
by GBuella, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D45984
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
GBuella updated this revision to Diff 143718.
https://reviews.llvm.org/D45984
Files:
docs/ClangCommandLineReference.rst
include/clang/Basic/BuiltinsX86.def
include/clang/Basic/BuiltinsX86_64.def
include/clang/Driver/Options.td
lib/Basic/Targets/X86.cpp
lib/Basic/Targets/X86.h
craig.topper added inline comments.
Comment at: lib/Headers/movdirintrin.h:39
+// Move quadword as direct store
+static __inline__ void
+__attribute__((__always_inline__, __nodebug__, __target__("movdiri")))
Sorry, the X86_64 preprocessor check should have
GBuella updated this revision to Diff 143647.
https://reviews.llvm.org/D45984
Files:
docs/ClangCommandLineReference.rst
include/clang/Basic/BuiltinsX86.def
include/clang/Basic/BuiltinsX86_64.def
include/clang/Driver/Options.td
lib/Basic/Targets/X86.cpp
lib/Basic/Targets/X86.h
GBuella updated this revision to Diff 143644.
https://reviews.llvm.org/D45984
Files:
docs/ClangCommandLineReference.rst
include/clang/Basic/BuiltinsX86.def
include/clang/Basic/BuiltinsX86_64.def
include/clang/Driver/Options.td
lib/Basic/Targets/X86.cpp
lib/Basic/Targets/X86.h
GBuella updated this revision to Diff 143642.
https://reviews.llvm.org/D45984
Files:
docs/ClangCommandLineReference.rst
include/clang/Basic/BuiltinsX86.def
include/clang/Basic/BuiltinsX86_64.def
include/clang/Driver/Options.td
lib/Basic/Targets/X86.cpp
lib/Basic/Targets/X86.h
craig.topper added inline comments.
Comment at: docs/ClangCommandLineReference.rst:2465
+.. option:: -mmovdiri, -mno-movdiri
+
Please alphabetize this correctly.
Comment at: include/clang/Basic/BuiltinsX86.def:1896
GBuella created this revision.
GBuella added a reviewer: craig.topper.
Herald added subscribers: cfe-commits, mgorny.
Repository:
rC Clang
https://reviews.llvm.org/D45984
Files:
docs/ClangCommandLineReference.rst
include/clang/Basic/BuiltinsX86.def
include/clang/Driver/Options.td