[PATCH] D122631: [scudo] Fix definition of canCache() function

Dominic Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 28 21:53:00 PDT 2022


ddcc created this revision.
ddcc added a reviewer: vitalybuka.
Herald added a project: All.
ddcc requested review of this revision.
Herald added a project: Sanitizers.

The called member function has boolean type, change this function to match


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D122631

Files:
  compiler-rt/lib/scudo/standalone/secondary.h


Index: compiler-rt/lib/scudo/standalone/secondary.h
===================================================================
--- compiler-rt/lib/scudo/standalone/secondary.h
+++ compiler-rt/lib/scudo/standalone/secondary.h
@@ -459,7 +459,7 @@
     }
   }
 
-  uptr canCache(uptr Size) { return Cache.canCache(Size); }
+  bool canCache(uptr Size) { return Cache.canCache(Size); }
 
   bool setOption(Option O, sptr Value) { return Cache.setOption(O, Value); }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D122631.418781.patch
Type: text/x-patch
Size: 456 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220329/5340efc5/attachment.bin>


More information about the llvm-commits mailing list