eugenis added a comment.
Merged. Please don't forget to update tests next time.
Repository:
rL LLVM
https://reviews.llvm.org/D44927
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
This revision was not accepted when it landed; it landed in state "Needs
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rL329241: Enable msan unconditionally on Linux. (authored by
eugenis, committed by ).
Herald added a subscriber: llvm-commit
vit9696 added a comment.
Now that https://reviews.llvm.org/D44926 was merged upstream, may I request
this to be merged as well?
As stated above it is pretty much pointless to have the check doubled, and
after more than a week nobody thought of any issue.
Thanks :)
Repository:
rC Clang
https
eugenis added a subscriber: kcc.
eugenis added a comment.
I too find these checks arbitrary and pointless. @kcc ?
Repository:
rC Clang
https://reviews.llvm.org/D44927
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/c
vit9696 created this revision.
vit9696 added a reviewer: eugenis.
Herald added a subscriber: cfe-commits.
Memeory sanitizer compatibility are already done in
MemorySanitizer::doInitialization. It verifies whether the necessary offsets
exist and bails out if not. For this reason it is no good to