[PATCH] D42639: [clang-move] Clever on handling header file which includes itself.

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 31 01:25:43 PST 2018


hokein marked an inline comment as done.
hokein added inline comments.


================
Comment at: clang-move/ClangMove.cpp:708
+    // Find old.h includes "old.h".
+    if (AbsoluteOldHeader == AbsoluteOldHeader) {
+      OldHeaderIncludeRangeInHeader = IncludeFilenameRange;
----------------
ioeric wrote:
> This check is always true?
Oops, good catch, should be AbsoluteIncludeHeader.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D42639





More information about the cfe-commits mailing list