[PATCH] D49986: [ADT] ImmutableList::add parameters are switched

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 30 07:04:14 PDT 2018


Szelethus created this revision.
Szelethus added reviewers: NoQ, george.karpenkov, dcoughlin, chandlerc.
Herald added a subscriber: cfe-commits.

Clang side changes, see https://reviews.llvm.org/D49985.


Repository:
  rC Clang

https://reviews.llvm.org/D49986

Files:
  include/clang/StaticAnalyzer/Core/PathSensitive/BasicValueFactory.h
  include/clang/StaticAnalyzer/Core/PathSensitive/ProgramStateTrait.h


Index: include/clang/StaticAnalyzer/Core/PathSensitive/ProgramStateTrait.h
===================================================================
--- include/clang/StaticAnalyzer/Core/PathSensitive/ProgramStateTrait.h
+++ include/clang/StaticAnalyzer/Core/PathSensitive/ProgramStateTrait.h
@@ -153,7 +153,7 @@
     using context_type = typename data_type::Factory &;
 
     static data_type Add(data_type L, key_type K, context_type F) {
-      return F.add(L, K);
+      return F.add(K, L);
     }
 
     static bool Contains(data_type L, key_type K) {
Index: include/clang/StaticAnalyzer/Core/PathSensitive/BasicValueFactory.h
===================================================================
--- include/clang/StaticAnalyzer/Core/PathSensitive/BasicValueFactory.h
+++ include/clang/StaticAnalyzer/Core/PathSensitive/BasicValueFactory.h
@@ -233,7 +233,7 @@
   }
 
   llvm::ImmutableList<SVal> prependSVal(SVal X, llvm::ImmutableList<SVal> L) {
-    return SValListFactory.add(L, X);
+    return SValListFactory.add(X, L);
   }
 
   llvm::ImmutableList<const CXXBaseSpecifier *> getEmptyCXXBaseList() {
@@ -243,7 +243,7 @@
   llvm::ImmutableList<const CXXBaseSpecifier *> prependCXXBase(
       const CXXBaseSpecifier *CBS,
       llvm::ImmutableList<const CXXBaseSpecifier *> L) {
-    return CXXBaseListFactory.add(L, CBS);
+    return CXXBaseListFactory.add(CBS, L);
   }
 
   const PointerToMemberData *accumCXXBase(


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D49986.157955.patch
Type: text/x-patch
Size: 1421 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180730/1006a507/attachment.bin>


More information about the cfe-commits mailing list