[PATCH] D60408: [LibTooling] Extend Transformer to support multiple simultaneous changes.

Tianle Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 19 20:12:44 PDT 2019


liutianle added a comment.

hi ymandel,

When I run "check-all", there are some warning/error in TransformerTest.cpp as follow. My version is llvm:0ee120077 and clang:d87ee8e678. Could you please have a fix or guild me how to fix it?
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

In file included from myLLVM/llvm/utils/unittest/googlemock/include/gmock/internal/gmock-internal-utils.h:47:0,

  from myLLVM/llvm/utils/unittest/googlemock/include/gmock/gmock-actions.h:46,
  from myLLVM/llvm/utils/unittest/googlemock/include/gmock/gmock.h:58,
  from myLLVM/llvm/tools/clang/unittests/Tooling/TransformerTest.cpp:13:

myLLVM/llvm/utils/unittest/googletest/include/gtest/gtest.h: In instantiation of 'testing::AssertionResult testing::internal::CmpHelperEQ(const char*, const char*, const T1&, const T2&) [with T1 = long unsigned int; T2 = int]':
myLLVM/llvm/utils/unittest/googletest/include/gtest/gtest.h:1421:23:   required from 'static testing::AssertionResult testing::internal::EqHelper<lhs_is_null_literal>::Compare(const char*, const char*, const T1&, const T2&) [with T1 = long unsigned int; T2 = int; bool lhs_is_null_literal = false]'
myLLVM/llvm/tools/clang/unittests/Tooling/TransformerTest.cpp:372:3:   required from here
myLLVM/llvm/utils/unittest/googletest/include/gtest/gtest.h:1392:11: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]

  if (lhs == rhs) {
          ^

------------------------------------------------------------------------------------------------------------------------------------------------------------------------------


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60408/new/

https://reviews.llvm.org/D60408





More information about the cfe-commits mailing list