No worries, I got it here:
dzur:~/sources/llvm/tools/clang> git svn dcommit Committing to https://llvm.org/svn/llvm-project/cfe/trunk ... M lib/Headers/f16cintrin.h Committed r260631 Thanks! -eric On Thu, Feb 11, 2016 at 2:20 PM Romanova, Katya < katya_roman...@playstation.sony.com> wrote: > Sure. I’ll change it. Do you mind if I do it when I’m back from vacation > (i.e. a couple of weeks from now)? > > I will send a patch. > > Thank you! > > Katya. > > > > *From:* Eric Christopher [mailto:echri...@gmail.com] > *Sent:* Wednesday, February 10, 2016 5:08 PM > > > *To:* Romanova, Katya; cfe-commits@lists.llvm.org > *Subject:* Re: r260333 - This patch adds doxygen comments for all the > intrinsincs in the header file f16cintrin.h. The doxygen comments are > automatically generated based on Sony's intrinsics document. > > > > > > On Wed, Feb 10, 2016 at 5:07 PM Joerg Sonnenberger < > jo...@britannica.bec.de> wrote: > > On Thu, Feb 11, 2016 at 12:18:49AM +0000, Eric Christopher via cfe-commits > wrote: > > Mostly that you didn't mention it in your commit message. > > > > I'd rather prefer the rest of the file be changed to match the __ rather > > than the other way around :) > > Depends. If it is a macro, not using __ is fine. If it is an inline > function, it *must* use the protected namespace, since an application can > #define a fancy junk here. > > > > Yep. Which is why I'd prefer the rest of the file changed to match the __. > I was just trying to get Katya to do it :) > > > > -eric >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
