Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(291)

Unified Diff: third_party/re2/README.chromium

Issue 149723005: Patch re2 to support MSan. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address eugenis's suggestions Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | third_party/re2/patches/re2-msan.patch » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/re2/README.chromium
diff --git a/third_party/re2/README.chromium b/third_party/re2/README.chromium
index 3617102dc9d5b2c00e450923f310bbadf35a9468..21c05a18ee0019cffad7bd5dadc75b5e24e1b85f 100644
--- a/third_party/re2/README.chromium
+++ b/third_party/re2/README.chromium
@@ -23,3 +23,7 @@ Local Modifications (to be applied in this order):
https://code.google.com/p/re2/issues/detail?id=76
- Memory optimization for filtered trees
(patches/re2-memory-optimization.patch)
+- Prevent unwanted reports from MemorySanitizer. Note: there's an upstream fix
+ for this (https://code.google.com/p/re2/issues/detail?id=77) which is rendered
+ ineffective by patches/remove-valgrind-code.patch
+ (patches/re2-msan.patch)
« no previous file with comments | « no previous file | third_party/re2/patches/re2-msan.patch » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698