SuccessChanges

Summary

  1. MemoryBuffer: Add a missing error-check to getOpenFileImpl (details)
  2. [InstCombine] Precommit test case for D66216 (details)
  3. [analyzer] Add docs for cplusplus.InnerPointer (details)
  4. [analyzer] Warn about -analyzer-configs being meant for development (details)
Commit 46bfdb956cb805d16388c142ee2872b20896e33b by pavel
MemoryBuffer: Add a missing error-check to getOpenFileImpl
Summary: In case the function was called with a desired read size *and*
the file was not an "mmap()" candidate, the function was falling back to
a
"pread()", but it was failing to check the result of that system call.
This meant that the function would return "success" even though the read
operation failed, and it returned a buffer full of uninitialized memory.
Reviewers: rnk, dblaikie
Subscribers: kristina, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D66224
llvm-svn: 368977
The file was modifiedllvm/unittests/Support/MemoryBufferTest.cpp (diff)
The file was modifiedllvm/lib/Support/MemoryBuffer.cpp (diff)
Commit 1bd898989ca9259ae0d2bd11eb204478987081c7 by flo
[InstCombine] Precommit test case for D66216
llvm-svn: 368978
The file was addedllvm/test/Transforms/InstCombine/phi-known-bits-operand-order.ll
Commit e6e133b7007635b7445c7d15806e9e418f38866c by dkszelethus
[analyzer] Add docs for cplusplus.InnerPointer
Differential Revision: https://reviews.llvm.org/D60281
llvm-svn: 368979
The file was modifiedclang/docs/analyzer/checkers.rst (diff)
Commit a1aef90bdd9ae7febce70f96f4a05a0074c8b556 by dkszelethus
[analyzer] Warn about -analyzer-configs being meant for development
purposes only
This is more of a temporary fix, long term, we should convert
AnalyzerOptions.def into the universally beloved (*coughs*) TableGen
format, where they can more easily be separated into developer-only,
alpha, and user-facing configs.
Differential Revision: https://reviews.llvm.org/D66261
llvm-svn: 368980
The file was modifiedclang/lib/StaticAnalyzer/Frontend/CheckerRegistration.cpp (diff)
The file was modifiedclang/include/clang/Driver/CC1Options.td (diff)