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

[Zachary Turner via llvm-commits]

I'm leaning towards bug in clang's optimizer.  I've scanned this code
multiple times and I really don't see any UB.  I ran it under ubsan on
Linux and it did not report any problems.

I guess maybe you could run it under ubsan on aarch64 as well?

Assuming that passes, where do we go from here?  I know 0 about the
optimizer or how to fix a bug in it, especially one that only occurs in a
backend I have no access to :-/

On Tue, Oct 18, 2016 at 2:40 AM Renato Golin <renato.golin at linaro.org>
wrote:

> 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161018/48c12da7/attachment.html>