admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert of Workaround for clang bug http://llvm.org/PR28348. (patchset #1 id:1 of https://codereview.webrtc.org/2110043003/ )

Browse Source 
Reason for revert:
Not needed since https://codereview.chromium.org/2110873002/.

Original issue's description:
> Workaround for clang bug http://llvm.org/PR28348.
>
> Permits rolling chromium further.
>
> BUG=
> TBR=tommi@webrtc.org
>
> Committed: f516585e10

TBR=tommi@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=

Review-Url: https://codereview.webrtc.org/2106383002
Cr-Commit-Position: refs/heads/master@{#13344}
This commit is contained in:
pbos 2016-06-30 02:32:32 -07:00 committed by Commit bot
parent ea41694e08
commit 9ef6785027
1 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
webrtc/media/base/videoframe_unittest.h
View File

@ -1216,10 +1216,7 @@ class VideoFrameTest : public testing::Test {
EXPECT_FALSE(expected_result); // NULL is okay if failure was expected.
return;
}
// TODO(pbos): Remove kPad once http://llvm.org/PR28348 is fixed and the fix
// is rolled in.
volatile const int kPad = 4095;
data_ptr += kPadToHeapSized + (-(static_cast<int>(data_size)) & kPad);
data_ptr += kPadToHeapSized + (-(static_cast<int>(data_size)) & 4095);
memcpy(data_ptr, sample, std::min(data_size, sample_size));
for (int i = 0; i < repeat_; ++i) {
EXPECT_EQ(expected_result, frame.Validate(fourcc, kWidth, kHeight,