[PATCH] D49588: [CStringSyntaxChecker] Fix build bot builds != x86 archs

Artem Dergachev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 20 10:58:47 PDT 2018


NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.

Looks good, please commit!

When you think that a fix for buildbots requires review, please revert first! Reviews have a much longer turnaround than acceptable buildbot downtime.

The new code does indeed look more correct, but yeah, i agree with @george.karpenkov that explaining what's going on when posting code on review is a good idea.


https://reviews.llvm.org/D49588





More information about the llvm-commits mailing list