[PATCH] D22111: [compiler-rt] Refactor the interception code on windows.
    Reid Kleckner via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Jul 11 15:38:54 PDT 2016
    
    
  
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
================
Comment at: lib/interception/interception_win.cc:274
@@ +273,3 @@
+static const int kMaxTrampolineRegion = 1024;
+static TrampolineMemoryRegion TrampolineRegions[kMaxTrampolineRegion];
+
----------------
OK, I believe that.
================
Comment at: lib/interception/tests/interception_win_test.cc:273
@@ +272,3 @@
+    const T &code,
+    TestOverrideFunction Override,
+    FunctionPrefixKind prefix_kind = FunctionPrefixNone) {
----------------
This should be 'override' instead of 'Override', since it's a variable. Also below.
http://reviews.llvm.org/D22111
    
    
More information about the llvm-commits
mailing list