[all-commits] [llvm/llvm-project] 9081d3: [clang-tidy] Make the cert/uppercase-literal-suffi...

Haojian Wu via All-commits all-commits at lists.llvm.org
Fri Jun 24 05:18:46 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9081d3d8097af5e89905faf0ed568ba7e848e8df
      https://github.com/llvm/llvm-project/commit/9081d3d8097af5e89905faf0ed568ba7e848e8df
  Author: Haojian Wu <hokein.wu at gmail.com>
  Date:   2022-06-24 (Fri, 24 Jun 2022)

  Changed paths:
    A clang-tools-extra/test/clang-tidy/checkers/Inputs/Headers/integral_constant.h
    M clang-tools-extra/test/clang-tidy/checkers/cert/uppercase-literal-suffix-integer.cpp
    M clang-tools-extra/test/clang-tidy/checkers/readability/uppercase-literal-suffix-float16.cpp
    M clang-tools-extra/test/clang-tidy/checkers/readability/uppercase-literal-suffix-floating-point.cpp
    M clang-tools-extra/test/clang-tidy/checkers/readability/uppercase-literal-suffix-hexadecimal-floating-point.cpp
    M clang-tools-extra/test/clang-tidy/checkers/readability/uppercase-literal-suffix-integer-custom-list.cpp
    M clang-tools-extra/test/clang-tidy/checkers/readability/uppercase-literal-suffix-integer-ms.cpp
    M clang-tools-extra/test/clang-tidy/checkers/readability/uppercase-literal-suffix-integer.cpp
    R clang-tools-extra/test/clang-tidy/checkers/readability/uppercase-literal-suffix.h

  Log Message:
  -----------
  [clang-tidy] Make the cert/uppercase-literal-suffix-integer fully hermetic.

after the test-reorg commit (89a1d03e2b379e325daa5249411e414bbd995b5e), the
cert/uppercase test starts to fail in our internal environment -- it accesses
a header file from "../readability", which is not friendly to a hermetic test environment.

This change makes the test fully hermetic, and does some cleanup on the
uppercase header (I think it is better to move it the share
Inputs/Header directory, and rename it)

Differential Revision: https://reviews.llvm.org/D128511




More information about the All-commits mailing list