New issue
Advanced search Search tips

Issue 694804 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Feb 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Enable asserts on llvm-next waterfall builders

Project Member Reported by manojgupta@chromium.org, Feb 21 2017

Issue description

More testing is required for llvm-next builds.

The clang error reported in https://bugs.chromium.org/p/chromium/issues/detail?id=690639
would have been caught early if llvm was build with asserts enabled.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 24 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/aaba78ccd1585b0a712f30d7877e6835625dd25c

commit aaba78ccd1585b0a712f30d7877e6835625dd25c
Author: Manoj Gupta <manojgupta@google.com>
Date: Fri Feb 24 23:30:47 2017

Enable asserts on llvm-next build.

Enabling asserts should help catch more errors for llvm-next
builders.

BUG= chromium:694804 
TEST=Building sys-libs/pam fails for veyron_jaq with llvm-next.

Change-Id: Iab6775823cce5b87ce89f5ed33aee0541f848b09
Reviewed-on: https://chromium-review.googlesource.com/445865
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Trybot-Ready: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Ting-Yuan Huang <laszio@chromium.org>

[rename] https://crrev.com/aaba78ccd1585b0a712f30d7877e6835625dd25c/sys-devel/llvm/llvm-4.0_pre285905-r6.ebuild

Project Member

Comment 2 by bugdroid1@chromium.org, Feb 24 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/aaba78ccd1585b0a712f30d7877e6835625dd25c

commit aaba78ccd1585b0a712f30d7877e6835625dd25c
Author: Manoj Gupta <manojgupta@google.com>
Date: Fri Feb 24 23:30:47 2017

Enable asserts on llvm-next build.

Enabling asserts should help catch more errors for llvm-next
builders.

BUG= chromium:694804 
TEST=Building sys-libs/pam fails for veyron_jaq with llvm-next.

Change-Id: Iab6775823cce5b87ce89f5ed33aee0541f848b09
Reviewed-on: https://chromium-review.googlesource.com/445865
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Trybot-Ready: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Ting-Yuan Huang <laszio@chromium.org>

[rename] https://crrev.com/aaba78ccd1585b0a712f30d7877e6835625dd25c/sys-devel/llvm/llvm-4.0_pre285905-r6.ebuild

Status: Fixed (was: Untriaged)
Status: Verified (was: Fixed)

Sign in to add a comment