commit 69aa1acdcb5a16491abf34cde26a546361c9988e
author Lei Zhang <thestig@chromium.org> Thu Feb 07 18:02:30 2019 +0000
committer Chromium commit bot <commit-bot@chromium.org> Thu Feb 07 18:02:30 2019 +0000
tree 4e4e07e7fb35eb5ff1fb1abb46d38c63bf6946d5
parent 762247be1246942790637504f6a726f522a3b399

Roll build and clang.

https://chromium.googlesource.com/chromium/src/build.git/+log/67630827e138..213e0a3a3d17
https://chromium.googlesource.com/chromium/src/tools/clang/+log/2cf76f56cb0f..3a16568a5648

Roll clang 349417:351477. They have to be rolled together due to the
Clang version bumper to 9.x.

Also update UnownedPtr to make its unit tests pass when
ProbeForLowSeverityLifetimeIssue() is active.

BUG=chromium:929475

Change-Id: I99d6d1fa6cc80e73dfd93e05f402aba1f9be1355
Reviewed-on: https://pdfium-review.googlesource.com/c/50170
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>

DEPS

diff --git a/DEPS b/DEPS
index 9b11795..f04b1d2 100644
--- a/DEPS
+++ b/DEPS
@@ -14,10 +14,10 @@
 
   'android_ndk_revision': '4e2cea441bfd43f0863d14f57b1e1844260b9884',
   'binutils_revision': '2be73f7fbf783d7a0b288e174a5773b67c7656bc',
-  'build_revision': '10ffa2f5e62cc90ac7af3e6b69225259dcbeba9d',
+  'build_revision': '213e0a3a3d17d89b6513181a28956166a9c2141c',
   'buildtools_revision': '2f02e1f363b1af2715536f38e239853f04ec1497',
   'catapult_revision': '7c1d51b169edfb62a3e2f88730f1182240cfe981',
-  'clang_revision': '2cf76f56cb0f9cb84105f00bbc742416729b905d',
+  'clang_revision': '3a16568a56486d7d032b8ec7b8dae892413a9a7a',
   'code_coverage_revision': '16298755a926f419babc4a11ccfeae0035f48c3f',
   'cygwin_revision': 'c89e446b273697fadf3a10ff1007a97c0b7de6df',
   'depot_tools_revision': 'ddbeac1c06a9c0ffa100f6f2af672af83f980fcb',

core/fxcrt/unowned_ptr.h

diff --git a/core/fxcrt/unowned_ptr.h b/core/fxcrt/unowned_ptr.h
index 0a44f9d..8be3bd3 100644
--- a/core/fxcrt/unowned_ptr.h
+++ b/core/fxcrt/unowned_ptr.h
@@ -107,7 +107,7 @@
   inline void ProbeForLowSeverityLifetimeIssue() {
 #if defined(MEMORY_TOOL_REPLACES_ALLOCATOR)
     if (m_pObj)
-      reinterpret_cast<const volatile uint8_t*>(m_pObj)[0];
+      (void)reinterpret_cast<const volatile uint8_t*>(m_pObj)[0];
 #endif
   }