[llvm] r284437 - Rename HexStyle -> HexFormatStyle, and remove a constexpr.

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 18 02:40:37 PDT 2016


On 18 October 2016 at 00:08, Zachary Turner via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> Author: zturner
> Date: Mon Oct 17 18:08:47 2016
> New Revision: 284437
>
> URL: http://llvm.org/viewvc/llvm-project?rev=284437&view=rev
> Log:
> Rename HexStyle -> HexFormatStyle, and remove a constexpr.
>
> This should fix the remaining broken builds.

Hi Zach,

Didn't quite fix all problems, but we're getting there... :)

http://lab.llvm.org:8011/builders/clang-cmake-aarch64-full/builds/3344

[ RUN      ] NativeFormatTest.IntegerTests
/home/buildslave/buildslave/clang-cmake-aarch64-full/llvm/unittests/Support/NativeFormatTests.cpp:249:
Failure
Value of: format_number(-10, IntegerStyle::Integer, 1)
  Actual: "-0"
Expected: "-10"
[  FAILED  ] NativeFormatTest.IntegerTests (0 ms)

I have reverted on r284462. Let me know if you need help testing.

cheers,
--renato


More information about the llvm-commits mailing list