[all-commits] [llvm/llvm-project] ea8e71: [clang][HeaderInsert] Do not treat defines with va...

kadir çetinkaya via All-commits all-commits at lists.llvm.org
Tue Aug 11 07:09:55 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: ea8e71c3da56f76ae2ab2f9ee7afc6444408ffab
      https://github.com/llvm/llvm-project/commit/ea8e71c3da56f76ae2ab2f9ee7afc6444408ffab
  Author: Kadir Cetinkaya <kadircet at google.com>
  Date:   2020-08-11 (Tue, 11 Aug 2020)

  Changed paths:
    M clang/lib/Tooling/Inclusions/HeaderIncludes.cpp
    M clang/unittests/Tooling/HeaderIncludesTest.cpp

  Log Message:
  -----------
  [clang][HeaderInsert] Do not treat defines with values as header guards

This was resulting in inserting headers at bogus locations, see
https://github.com/ycm-core/YouCompleteMe/issues/3736 for an example.

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




More information about the All-commits mailing list