Hahnfeld added a comment.
In https://reviews.llvm.org/D46540#1092620, @GBuella wrote:
> In https://reviews.llvm.org/D46540#1091625, @Hahnfeld wrote:
>
> > Could you maybe add some short summaries to your patches? It's hard for
> > non-Intel employees to guess what all these instructions do...
>
This revision was automatically updated to reflect the committed changes.
Closed by commit rC331962: [X86] ptwrite intrinsic (authored by GBuella,
committed by ).
Changed prior to commit:
https://reviews.llvm.org/D46540?vs=145681&id=146081#toc
Repository:
rC Clang
https://reviews.llvm.org/D
GBuella added a comment.
In https://reviews.llvm.org/D46540#1091625, @Hahnfeld wrote:
> Could you maybe add some short summaries to your patches? It's hard for
> non-Intel employees to guess what all these instructions do...
Well, I was thinking I could copy-paste this from
https://software.i
Hahnfeld added a comment.
Could you maybe add some short summaries to your patches? It's hard for
non-Intel employees to guess what all these instructions do...
https://reviews.llvm.org/D46540
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D46540
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/li
GBuella updated this revision to Diff 145681.
GBuella added a comment.
Rebased.
https://reviews.llvm.org/D46540
Files:
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
lib/Hea
GBuella created this revision.
GBuella added a reviewer: craig.topper.
Herald added subscribers: cfe-commits, mgorny.
Repository:
rC Clang
https://reviews.llvm.org/D46540
Files:
include/clang/Basic/BuiltinsX86.def
include/clang/Basic/BuiltinsX86_64.def
include/clang/Driver/Options.td
l
