[PATCH] D64790: [LLVM][NFC] Adding an Alignment type to LLVM

Yvan Roux via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 1 02:16:55 PDT 2019


On Thu, 1 Aug 2019 at 11:14, Guillaume Chatelet <gchatelet at google.com> wrote:
>
> Thx Yvan. I believe https://reviews.llvm.org/D65563 will fix it (mismatch between size_t and uint64_t).
> I'll go ahead and submit it if you don't mind.

Indeed, I approved it, Thanks Guillaume

> On Thu, Aug 1, 2019 at 10:38 AM Yvan Roux <yvan.roux at linaro.org> wrote:
>>
>> Hi Guillaume,
>>
>> ARM bots are still broken (last build is based on r367519) logs are
>> available here:
>>
>> http://lab.llvm.org:8011/builders/clang-cmake-armv7-quick/builds/9390/steps/ninja%20check%201/logs/FAIL%3A%20LLVM-Unit%3A%3AAlignmentDeathTest.CompareAlignToUndefMaybeAlign
>>
>> http://lab.llvm.org:8011/waterfall?show=clang-cmake-armv7-quick
>>
>> Cheers,
>> Yvan
>>
>> On Thu, 1 Aug 2019 at 09:30, Guillaume Chatelet via Phabricator via
>> llvm-commits <llvm-commits at lists.llvm.org> wrote:
>> >
>> > gchatelet marked 2 inline comments as done.
>> > gchatelet added a comment.
>> >
>> > @aprantl I've created https://reviews.llvm.org/D65558 to address your comments since this patch is already submitted.
>> >
>> >
>> > Repository:
>> >   rL LLVM
>> >
>> > CHANGES SINCE LAST ACTION
>> >   https://reviews.llvm.org/D64790/new/
>> >
>> > https://reviews.llvm.org/D64790
>> >
>> >
>> >
>> > _______________________________________________
>> > llvm-commits mailing list
>> > llvm-commits at lists.llvm.org
>> > https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits


More information about the llvm-commits mailing list