This is an archive of the discontinued LLVM Phabricator instance.

Remove trailing spaces in x86 intrinsic headers
ClosedPublic

Authored by kromanova on May 25 2016, 1:02 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

kromanova updated this revision to Diff 58389.May 25 2016, 1:02 AM
kromanova retitled this revision from to Remove trailing spaces in x86 intrinsic headers.
kromanova updated this object.
kromanova added a reviewer: m_zuckerman.
kromanova set the repository for this revision to rL LLVM.
kromanova added a subscriber: cfe-commits.
m_zuckerman edited edge metadata.May 25 2016, 4:21 AM

First thanks

I don't see any problem with the patch.
but if you can please add full svn diff from clang.

svn diff --diff-cmd=diff -x -U999999 > x.patch

kromanova updated this revision to Diff 58472.May 25 2016, 12:42 PM
kromanova edited edge metadata.

I attached full svn diff. Thank you Michael!

I had to clean up the trailing spaces from doxygen comments (because it mess up our post-processing scripts), but since I was doing it, I decided to clean up the rest of the header files in the intrinsics header directory. That's why some of the files that you recently touched were affected.

m_zuckerman accepted this revision.May 27 2016, 2:23 AM
m_zuckerman edited edge metadata.

LGTM

This revision is now accepted and ready to land.May 27 2016, 2:23 AM
This revision was automatically updated to reflect the committed changes.