[PATCH] D28832: Improve redefinition errors pointing to the same header.

2017-01-18 Thread Vassil Vassilev via Phabricator via cfe-commits
v.g.vassilev added a comment. This looks good to me. https://reviews.llvm.org/D28832 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r292355 - Revert "[xray] try to fix thumb buildbot"

2017-01-18 Thread Renato Golin via cfe-commits
Author: rengolin Date: Wed Jan 18 03:05:32 2017 New Revision: 292355 URL: http://llvm.org/viewvc/llvm-project?rev=292355&view=rev Log: Revert "[xray] try to fix thumb buildbot" This reverts commit r292268, as it didn't fix the buildbots. Modified: cfe/trunk/lib/Driver/Tools.cpp Modified: cf

[PATCH] D28849: [compiler-rt] [test] Fix page address logic in clear_cache_test to use binary negation

2017-01-18 Thread Michał Górny via Phabricator via cfe-commits
mgorny created this revision. Herald added a subscriber: dberris. Fix the logic used to calculate page address in clear_cache_test to use the binary negation of 4095 rather than arithmetic. The latter gives incorrect result since: -4095 -> 0xf001 ~4095 -> 0xf000 Alternatively, -4096

[libcxx] r292354 - Merge R292276: Fix std::string assignment ambiguity from braced initializer lists.

2017-01-18 Thread Eric Fiselier via cfe-commits
Author: ericwf Date: Wed Jan 18 02:43:24 2017 New Revision: 292354 URL: http://llvm.org/viewvc/llvm-project?rev=292354&view=rev Log: Merge R292276: Fix std::string assignment ambiguity from braced initializer lists. When support for `basic_string_view` was added to string it also added new assig

[libcxx] r292351 - Re-add mechanism to override LIT options using enviroment variables.

2017-01-18 Thread Eric Fiselier via cfe-commits
Author: ericwf Date: Wed Jan 18 01:56:57 2017 New Revision: 292351 URL: http://llvm.org/viewvc/llvm-project?rev=292351&view=rev Log: Re-add mechanism to override LIT options using enviroment variables. There was a dumb mistake in the original commit that has now been fixed Modified: libcxx/t

[PATCH] D27202: [analyzer] Do not conjure a symbol for return value of a conservatively evaluated function

2017-01-18 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ planned changes to this revision. NoQ added a comment. Yep, because there are a lot more places in which the value of the expression suddenly changes, i think i'd want to either fix all of them, or (if i find some of those really reasonable) make a callback for checkers to subscribe to valu

<    1   2