> On May 18, 2018, at 11:48 AM, Eric Liu <ioe...@google.com> wrote:
> 
> So I have reverted this with r332751.

Thanks!


> I can't see how this introduced cyclic dependencies in module build, as the 
> dependencies should be clangTooling -> clangFormat -> clangToolingInclusions. 
> I'm wondering if there is any module configurations that I need to update to 
> make this work. Right now, module doesn't seem to make any difference between 
> clangTooling and clangToolingInclusions... I'd appreciate it if someone who 
> knows how clang module build is set up could help take a look.

+ Bruno & David who have more experience in this area than I do.


> Unfortunately, I couldn't reproduce this on my workstation as the module 
> build seemed to be broken due to other errors.

The build is dependent on having a set of modularized system headers -- perhaps 
that's the issue.


> I was seeing a lot of warnings; the clang I used to build llvm was built near 
> HEAD. Maybe my clang is too new?

This sounds a bit suspicious. Can new clang diagnostics land without clean 
stage2 builds? If so, we can look into enabling -Werror on our stage2 bots to 
catch this sort of thing earlier.

vedant

> 
> Thanks,
> Eric
> 
> On Fri, May 18, 2018 at 7:52 PM Eric Liu <ioe...@google.com 
> <mailto:ioe...@google.com>> wrote:
> Sorry, I'll look into it right now. Will revert it if I can't find a quick 
> fix.
> 
> On Fri, May 18, 2018, 19:49 Amara Emerson <aemer...@apple.com 
> <mailto:aemer...@apple.com>> wrote:
> Hi Eric,
> 
> Green dragon buildbots have started failing too, e.g.: 
> http://green.lab.llvm.org/green/job/clang-stage2-Rthinlto/10222/ 
> <http://green.lab.llvm.org/green/job/clang-stage2-Rthinlto/10222/>
> 
> If you don’t have a quick fix can you please revert it.
> 
> Thanks,
> Amara
> 
> 
>> On May 18, 2018, at 7:46 PM, Eric Liu <ioe...@google.com 
>> <mailto:ioe...@google.com>> wrote:
>> 
>> Hi Vedant,
>> 
>> It seems that your build was not using cmake files in the source tree? 
>> lib/Tooling/Inclusions/ is a (new) standalone library 
>> (clangToolingInclusions, similar to clangToolingCore). You might need update 
>> your build to reflect this change. Let me know if you have any further 
>> question.
>> 
>> Thanks,
>> Eric
>> 
>> On Fri, May 18, 2018 at 6:41 PM Vedant Kumar <v...@apple.com 
>> <mailto:v...@apple.com>> wrote:
>> Hi Eric,
>> 
>> I think there might be a cyclic dependency issue here, possibly one that's 
>> only visible with LLVM_ENABLE_MODULES=On. I see:
>> 
>> While building module 'Clang_Tooling' imported from /Users/vsk/src/llvm.org 
>> <http://llvm.org/>-master/llvm/tools/clang/lib/Tooling/Execution.cpp:10:     
>>                         
>> While building module 'Clang_Format' imported from /Users/vsk/src/llvm.org 
>> <http://llvm.org/>-master/llvm/tools/clang/include/clang/Tooling/Refactoring/AtomicChange.h:19:
>>        
>> In file included from <module-includes>:1:                                   
>>                                                                              
>>    
>> /Users/vsk/src/llvm.org 
>> <http://llvm.org/>-master/llvm/tools/clang/include/clang/Format/Format.h:20:10:
>>  fatal error: cyclic dependency in module 'Clang_Tooling': Clang_Tooling $> 
>> Clang_Format -> Clang_Tooling                                                
>>                                                     
>> #include "clang/Tooling/Inclusions/IncludeStyle.h"
>>          ^                          
>> While building module 'Clang_Tooling' imported from /Users/vsk/src/llvm.org 
>> <http://llvm.org/>-master/llvm/tools/clang/lib/Tooling/Execution.cpp:10:
>> In file included from <module-includes>:22:                                  
>>                         
>> In file included from /Users/vsk/src/llvm.org 
>> <http://llvm.org/>-master/llvm/tools/clang/include/clang/Tooling/Refactoring/RefactoringAction.h:14:
>> In file included from /Users/vsk/src/llvm.org 
>> <http://llvm.org/>-master/llvm/tools/clang/include/clang/Tooling/Refactoring/RefactoringActionRules.h:14:
>> In file included from /Users/vsk/src/llvm.org 
>> <http://llvm.org/>-master/llvm/tools/clang/include/clang/Tooling/Refactoring/RefactoringActionRulesInternal.h:16:
>> In file included from /Users/vsk/src/llvm.org 
>> <http://llvm.org/>-master/llvm/tools/clang/include/clang/Tooling/Refactoring/RefactoringResultConsumer.h:14:
>> /Users/vsk/src/llvm.org 
>> <http://llvm.org/>-master/llvm/tools/clang/include/clang/Tooling/Refactoring/AtomicChange.h:19:10:
>>  fatal error: could not build module 'Clang_Format'
>> #include "clang/Format/Format.h"
>>  ~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~
>> /Users/vsk/src/llvm.org 
>> <http://llvm.org/>-master/llvm/tools/clang/lib/Tooling/Execution.cpp:10:10: 
>> fatal error: could not build module 'Clang_Tooling'
>> #include "clang/Tooling/Execution.h"                                         
>>    
>>  ~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~                            
>> 3 errors generated.
>> 
>> Could you take a look?
>> 
>> thanks,
>> vedant
>> 
>> > On May 18, 2018, at 7:16 AM, Eric Liu via cfe-commits 
>> > <cfe-commits@lists.llvm.org <mailto:cfe-commits@lists.llvm.org>> wrote:
>> > 
>> > Author: ioeric
>> > Date: Fri May 18 07:16:37 2018
>> > New Revision: 332720
>> > 
>> > URL: http://llvm.org/viewvc/llvm-project?rev=332720&view=rev 
>> > <http://llvm.org/viewvc/llvm-project?rev=332720&view=rev>
>> > Log:
>> > Move #include manipulation code to new lib/Tooling/Inclusions.
>> > 
>> > Summary:
>> > clangToolingCore is linked into almost everything (incl. clang), but
>> > not few tools need #include manipulation at this point. So pull this into a
>> > separate library in Tooling.
>> > 
>> > Reviewers: ilya-biryukov
>> > 
>> > Subscribers: klimek, mgorny, cfe-commits, thakis
>> > 
>> > Differential Revision: https://reviews.llvm.org/D47068 
>> > <https://reviews.llvm.org/D47068>
>> > 
>> > Added:
>> >    cfe/trunk/include/clang/Tooling/Inclusions/
>> >    cfe/trunk/include/clang/Tooling/Inclusions/HeaderIncludes.h
>> >      - copied, changed from r332717, 
>> > cfe/trunk/include/clang/Tooling/Core/HeaderIncludes.h
>> >    cfe/trunk/include/clang/Tooling/Inclusions/IncludeStyle.h
>> >      - copied, changed from r332717, 
>> > cfe/trunk/include/clang/Tooling/Core/IncludeStyle.h
>> >    cfe/trunk/lib/Tooling/Inclusions/
>> >    cfe/trunk/lib/Tooling/Inclusions/CMakeLists.txt
>> >      - copied, changed from r332717, 
>> > cfe/trunk/lib/Tooling/Core/CMakeLists.txt
>> >    cfe/trunk/lib/Tooling/Inclusions/HeaderIncludes.cpp
>> >      - copied, changed from r332717, 
>> > cfe/trunk/lib/Tooling/Core/HeaderIncludes.cpp
>> >    cfe/trunk/lib/Tooling/Inclusions/IncludeStyle.cpp
>> >      - copied, changed from r332717, 
>> > cfe/trunk/lib/Tooling/Core/IncludeStyle.cpp
>> > Removed:
>> >    cfe/trunk/include/clang/Tooling/Core/HeaderIncludes.h
>> >    cfe/trunk/include/clang/Tooling/Core/IncludeStyle.h
>> >    cfe/trunk/lib/Tooling/Core/HeaderIncludes.cpp
>> >    cfe/trunk/lib/Tooling/Core/IncludeStyle.cpp
>> > Modified:
>> >    cfe/trunk/include/clang/Format/Format.h
>> >    cfe/trunk/lib/Format/CMakeLists.txt
>> >    cfe/trunk/lib/Format/Format.cpp
>> >    cfe/trunk/lib/Tooling/CMakeLists.txt
>> >    cfe/trunk/lib/Tooling/Core/CMakeLists.txt
>> >    cfe/trunk/unittests/Tooling/HeaderIncludesTest.cpp
>> > 
>> > Modified: cfe/trunk/include/clang/Format/Format.h
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Format/Format.h?rev=332720&r1=332719&r2=332720&view=diff
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Format/Format.h?rev=332720&r1=332719&r2=332720&view=diff>
>> > ==============================================================================
>> > --- cfe/trunk/include/clang/Format/Format.h (original)
>> > +++ cfe/trunk/include/clang/Format/Format.h Fri May 18 07:16:37 2018
>> > @@ -16,8 +16,8 @@
>> > #define LLVM_CLANG_FORMAT_FORMAT_H
>> > 
>> > #include "clang/Basic/LangOptions.h"
>> > -#include "clang/Tooling/Core/IncludeStyle.h"
>> > #include "clang/Tooling/Core/Replacement.h"
>> > +#include "clang/Tooling/Inclusions/IncludeStyle.h"
>> > #include "llvm/ADT/ArrayRef.h"
>> > #include "llvm/Support/Regex.h"
>> > #include <system_error>
>> > 
>> > Removed: cfe/trunk/include/clang/Tooling/Core/HeaderIncludes.h
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Tooling/Core/HeaderIncludes.h?rev=332719&view=auto
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Tooling/Core/HeaderIncludes.h?rev=332719&view=auto>
>> > ==============================================================================
>> > --- cfe/trunk/include/clang/Tooling/Core/HeaderIncludes.h (original)
>> > +++ cfe/trunk/include/clang/Tooling/Core/HeaderIncludes.h (removed)
>> > @@ -1,137 +0,0 @@
>> > -//===--- HeaderIncludes.h - Insert/Delete #includes for C++ code--*- 
>> > C++-*-===//
>> > -//
>> > -//                     The LLVM Compiler Infrastructure
>> > -//
>> > -// This file is distributed under the University of Illinois Open Source
>> > -// License. See LICENSE.TXT for details.
>> > -//
>> > -//===----------------------------------------------------------------------===//
>> > -
>> > -#ifndef LLVM_CLANG_TOOLING_CORE_HEADERINCLUDES_H
>> > -#define LLVM_CLANG_TOOLING_CORE_HEADERINCLUDES_H
>> > -
>> > -#include "clang/Basic/SourceManager.h"
>> > -#include "clang/Tooling/Core/Replacement.h"
>> > -#include "clang/Tooling/Core/IncludeStyle.h"
>> > -#include "llvm/Support/Path.h"
>> > -#include "llvm/Support/Regex.h"
>> > -#include <unordered_map>
>> > -
>> > -namespace clang {
>> > -namespace tooling {
>> > -
>> > -/// This class manages priorities of C++ #include categories and 
>> > calculates
>> > -/// priorities for headers.
>> > -/// FIXME(ioeric): move this class into implementation file when 
>> > clang-format's
>> > -/// include sorting functions are also moved here.
>> > -class IncludeCategoryManager {
>> > -public:
>> > -  IncludeCategoryManager(const IncludeStyle &Style, StringRef FileName);
>> > -
>> > -  /// Returns the priority of the category which \p IncludeName belongs 
>> > to.
>> > -  /// If \p CheckMainHeader is true and \p IncludeName is a main header, 
>> > returns
>> > -  /// 0. Otherwise, returns the priority of the matching category or 
>> > INT_MAX.
>> > -  /// NOTE: this API is not thread-safe!
>> > -  int getIncludePriority(StringRef IncludeName, bool CheckMainHeader) 
>> > const;
>> > -
>> > -private:
>> > -  bool isMainHeader(StringRef IncludeName) const;
>> > -
>> > -  const IncludeStyle Style;
>> > -  bool IsMainFile;
>> > -  std::string FileName;
>> > -  // This refers to a substring in FileName.
>> > -  StringRef FileStem;
>> > -  // Regex is not thread-safe.
>> > -  mutable SmallVector<llvm::Regex, 4> CategoryRegexs;
>> > -};
>> > -
>> > -/// Generates replacements for inserting or deleting #include directives 
>> > in a
>> > -/// file.
>> > -class HeaderIncludes {
>> > -public:
>> > -  HeaderIncludes(llvm::StringRef FileName, llvm::StringRef Code,
>> > -                 const IncludeStyle &Style);
>> > -
>> > -  /// Inserts an #include directive of \p Header into the code. If \p 
>> > IsAngled
>> > -  /// is true, \p Header will be quoted with <> in the directive; 
>> > otherwise, it
>> > -  /// will be quoted with "".
>> > -  ///
>> > -  /// When searching for points to insert new header, this ignores 
>> > #include's
>> > -  /// after the #include block(s) in the beginning of a file to avoid 
>> > inserting
>> > -  /// headers into code sections where new #include's should not be added 
>> > by
>> > -  /// default. These code sections include:
>> > -  ///   - raw string literals (containing #include).
>> > -  ///   - #if blocks.
>> > -  ///   - Special #include's among declarations (e.g. functions).
>> > -  ///
>> > -  /// Returns a replacement that inserts the new header into a suitable 
>> > #include
>> > -  /// block of the same category. This respects the order of the existing
>> > -  /// #includes in the block; if the existing #includes are not already 
>> > sorted,
>> > -  /// this will simply insert the #include in front of the first #include 
>> > of the
>> > -  /// same category in the code that should be sorted after \p 
>> > IncludeName. If
>> > -  /// \p IncludeName already exists (with exactly the same spelling), this
>> > -  /// returns None.
>> > -  llvm::Optional<tooling::Replacement> insert(llvm::StringRef Header,
>> > -                                              bool IsAngled) const;
>> > -
>> > -  /// Removes all existing #includes of \p Header quoted with <> if \p 
>> > IsAngled
>> > -  /// is true or "" if \p IsAngled is false.
>> > -  /// This doesn't resolve the header file path; it only deletes 
>> > #includes with
>> > -  /// exactly the same spelling.
>> > -  tooling::Replacements remove(llvm::StringRef Header, bool IsAngled) 
>> > const;
>> > -
>> > -private:
>> > -  struct Include {
>> > -    Include(StringRef Name, tooling::Range R) : Name(Name), R(R) {}
>> > -
>> > -    // An include header quoted with either <> or "".
>> > -    std::string Name;
>> > -    // The range of the whole line of include directive including any 
>> > eading
>> > -    // whitespaces and trailing comment.
>> > -    tooling::Range R;
>> > -  };
>> > -
>> > -  void addExistingInclude(Include IncludeToAdd, unsigned NextLineOffset);
>> > -
>> > -  std::string FileName;
>> > -  std::string Code;
>> > -
>> > -  // Map from include name (quotation trimmed) to a list of existing 
>> > includes
>> > -  // (in case there are more than one) with the name in the current file. 
>> > <x>
>> > -  // and "x" will be treated as the same header when deleting #includes.
>> > -  llvm::StringMap<llvm::SmallVector<Include, 1>> ExistingIncludes;
>> > -
>> > -  /// Map from priorities of #include categories to all #includes in the 
>> > same
>> > -  /// category. This is used to find #includes of the same category when
>> > -  /// inserting new #includes. #includes in the same categories are 
>> > sorted in
>> > -  /// in the order they appear in the source file.
>> > -  /// See comment for "FormatStyle::IncludeCategories" for details about 
>> > include
>> > -  /// priorities.
>> > -  std::unordered_map<int, llvm::SmallVector<const Include *, 8>>
>> > -      IncludesByPriority;
>> > -
>> > -  int FirstIncludeOffset;
>> > -  // All new headers should be inserted after this offset (e.g. after 
>> > header
>> > -  // guards, file comment).
>> > -  unsigned MinInsertOffset;
>> > -  // Max insertion offset in the original code. For example, we want to 
>> > avoid
>> > -  // inserting new #includes into the actual code section (e.g. after a
>> > -  // declaration).
>> > -  unsigned MaxInsertOffset;
>> > -  IncludeCategoryManager Categories;
>> > -  // Record the offset of the end of the last include in each category.
>> > -  std::unordered_map<int, int> CategoryEndOffsets;
>> > -
>> > -  // All possible priorities.
>> > -  std::set<int> Priorities;
>> > -
>> > -  // Matches a whole #include directive.
>> > -  llvm::Regex IncludeRegex;
>> > -};
>> > -
>> > -
>> > -} // namespace tooling
>> > -} // namespace clang
>> > -
>> > -#endif // LLVM_CLANG_TOOLING_CORE_HEADERINCLUDES_H
>> > 
>> > Removed: cfe/trunk/include/clang/Tooling/Core/IncludeStyle.h
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Tooling/Core/IncludeStyle.h?rev=332719&view=auto
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Tooling/Core/IncludeStyle.h?rev=332719&view=auto>
>> > ==============================================================================
>> > --- cfe/trunk/include/clang/Tooling/Core/IncludeStyle.h (original)
>> > +++ cfe/trunk/include/clang/Tooling/Core/IncludeStyle.h (removed)
>> > @@ -1,133 +0,0 @@
>> > -//===--- IncludeStyle.h - Style of C++ #include directives -------*- 
>> > C++-*-===//
>> > -//
>> > -//                     The LLVM Compiler Infrastructure
>> > -//
>> > -// This file is distributed under the University of Illinois Open Source
>> > -// License. See LICENSE.TXT for details.
>> > -//
>> > -//===----------------------------------------------------------------------===//
>> > -
>> > -#ifndef LLVM_CLANG_TOOLING_CORE_INCLUDESTYLE_H
>> > -#define LLVM_CLANG_TOOLING_CORE_INCLUDESTYLE_H
>> > -
>> > -#include "llvm/Support/YAMLTraits.h"
>> > -#include <string>
>> > -#include <vector>
>> > -
>> > -namespace clang {
>> > -namespace tooling {
>> > -
>> > -/// Style for sorting and grouping C++ #include directives.
>> > -struct IncludeStyle {
>> > -  /// Styles for sorting multiple ``#include`` blocks.
>> > -  enum IncludeBlocksStyle {
>> > -    /// Sort each ``#include`` block separately.
>> > -    /// \code
>> > -    ///    #include "b.h"               into      #include "b.h"
>> > -    ///
>> > -    ///    #include <lib/main.h>                  #include "a.h"
>> > -    ///    #include "a.h"                         #include <lib/main.h>
>> > -    /// \endcode
>> > -    IBS_Preserve,
>> > -    /// Merge multiple ``#include`` blocks together and sort as one.
>> > -    /// \code
>> > -    ///    #include "b.h"               into      #include "a.h"
>> > -    ///                                           #include "b.h"
>> > -    ///    #include <lib/main.h>                  #include <lib/main.h>
>> > -    ///    #include "a.h"
>> > -    /// \endcode
>> > -    IBS_Merge,
>> > -    /// Merge multiple ``#include`` blocks together and sort as one.
>> > -    /// Then split into groups based on category priority. See
>> > -    /// ``IncludeCategories``.
>> > -    /// \code
>> > -    ///    #include "b.h"               into      #include "a.h"
>> > -    ///                                           #include "b.h"
>> > -    ///    #include <lib/main.h>
>> > -    ///    #include "a.h"                         #include <lib/main.h>
>> > -    /// \endcode
>> > -    IBS_Regroup,
>> > -  };
>> > -
>> > -  /// Dependent on the value, multiple ``#include`` blocks can be sorted
>> > -  /// as one and divided based on category.
>> > -  IncludeBlocksStyle IncludeBlocks;
>> > -
>> > -  /// See documentation of ``IncludeCategories``.
>> > -  struct IncludeCategory {
>> > -    /// The regular expression that this category matches.
>> > -    std::string Regex;
>> > -    /// The priority to assign to this category.
>> > -    int Priority;
>> > -    bool operator==(const IncludeCategory &Other) const {
>> > -      return Regex == Other.Regex && Priority == Other.Priority;
>> > -    }
>> > -  };
>> > -
>> > -  /// Regular expressions denoting the different ``#include`` categories
>> > -  /// used for ordering ``#includes``.
>> > -  ///
>> > -  /// These regular expressions are matched against the filename of an 
>> > include
>> > -  /// (including the <> or "") in order. The value belonging to the first
>> > -  /// matching regular expression is assigned and ``#includes`` are 
>> > sorted first
>> > -  /// according to increasing category number and then alphabetically 
>> > within
>> > -  /// each category.
>> > -  ///
>> > -  /// If none of the regular expressions match, INT_MAX is assigned as
>> > -  /// category. The main header for a source file automatically gets 
>> > category 0.
>> > -  /// so that it is generally kept at the beginning of the ``#includes``
>> > -  /// (http://llvm.org/docs/CodingStandards.html#include-style 
>> > <http://llvm.org/docs/CodingStandards.html#include-style>). However, you
>> > -  /// can also assign negative priorities if you have certain headers that
>> > -  /// always need to be first.
>> > -  ///
>> > -  /// To configure this in the .clang-format file, use:
>> > -  /// \code{.yaml}
>> > -  ///   IncludeCategories:
>> > -  ///     - Regex:           '^"(llvm|llvm-c|clang|clang-c)/'
>> > -  ///       Priority:        2
>> > -  ///     - Regex:           '^(<|"(gtest|gmock|isl|json)/)'
>> > -  ///       Priority:        3
>> > -  ///     - Regex:           '.*'
>> > -  ///       Priority:        1
>> > -  /// \endcode
>> > -  std::vector<IncludeCategory> IncludeCategories;
>> > -
>> > -  /// Specify a regular expression of suffixes that are allowed in the
>> > -  /// file-to-main-include mapping.
>> > -  ///
>> > -  /// When guessing whether a #include is the "main" include (to assign
>> > -  /// category 0, see above), use this regex of allowed suffixes to the 
>> > header
>> > -  /// stem. A partial match is done, so that:
>> > -  /// - "" means "arbitrary suffix"
>> > -  /// - "$" means "no suffix"
>> > -  ///
>> > -  /// For example, if configured to "(_test)?$", then a header a.h would 
>> > be seen
>> > -  /// as the "main" include in both a.cc <http://a.cc/> and a_test.cc 
>> > <http://a_test.cc/>.
>> > -  std::string IncludeIsMainRegex;
>> > -};
>> > -
>> > -} // namespace tooling
>> > -} // namespace clang
>> > -
>> > -LLVM_YAML_IS_SEQUENCE_VECTOR(clang::tooling::IncludeStyle::IncludeCategory)
>> > -
>> > -namespace llvm {
>> > -namespace yaml {
>> > -
>> > -template <>
>> > -struct MappingTraits<clang::tooling::IncludeStyle::IncludeCategory> {
>> > -  static void mapping(IO &IO,
>> > -                      clang::tooling::IncludeStyle::IncludeCategory 
>> > &Category);
>> > -};
>> > -
>> > -template <>
>> > -struct ScalarEnumerationTraits<
>> > -    clang::tooling::IncludeStyle::IncludeBlocksStyle> {
>> > -  static void
>> > -  enumeration(IO &IO, clang::tooling::IncludeStyle::IncludeBlocksStyle 
>> > &Value);
>> > -};
>> > -
>> > -} // namespace yaml
>> > -} // namespace llvm
>> > -
>> > -#endif // LLVM_CLANG_TOOLING_CORE_INCLUDESTYLE_H
>> > 
>> > Copied: cfe/trunk/include/clang/Tooling/Inclusions/HeaderIncludes.h (from 
>> > r332717, cfe/trunk/include/clang/Tooling/Core/HeaderIncludes.h)
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Tooling/Inclusions/HeaderIncludes.h?p2=cfe/trunk/include/clang/Tooling/Inclusions/HeaderIncludes.h&p1=cfe/trunk/include/clang/Tooling/Core/HeaderIncludes.h&r1=332717&r2=332720&rev=332720&view=diff
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Tooling/Inclusions/HeaderIncludes.h?p2=cfe/trunk/include/clang/Tooling/Inclusions/HeaderIncludes.h&p1=cfe/trunk/include/clang/Tooling/Core/HeaderIncludes.h&r1=332717&r2=332720&rev=332720&view=diff>
>> > ==============================================================================
>> > --- cfe/trunk/include/clang/Tooling/Core/HeaderIncludes.h (original)
>> > +++ cfe/trunk/include/clang/Tooling/Inclusions/HeaderIncludes.h Fri May 18 
>> > 07:16:37 2018
>> > @@ -7,12 +7,12 @@
>> > //
>> > //===----------------------------------------------------------------------===//
>> > 
>> > -#ifndef LLVM_CLANG_TOOLING_CORE_HEADERINCLUDES_H
>> > -#define LLVM_CLANG_TOOLING_CORE_HEADERINCLUDES_H
>> > +#ifndef LLVM_CLANG_TOOLING_INCLUSIONS_HEADERINCLUDES_H
>> > +#define LLVM_CLANG_TOOLING_INCLUSIONS_HEADERINCLUDES_H
>> > 
>> > #include "clang/Basic/SourceManager.h"
>> > #include "clang/Tooling/Core/Replacement.h"
>> > -#include "clang/Tooling/Core/IncludeStyle.h"
>> > +#include "clang/Tooling/Inclusions/IncludeStyle.h"
>> > #include "llvm/Support/Path.h"
>> > #include "llvm/Support/Regex.h"
>> > #include <unordered_map>
>> > @@ -134,4 +134,4 @@ private:
>> > } // namespace tooling
>> > } // namespace clang
>> > 
>> > -#endif // LLVM_CLANG_TOOLING_CORE_HEADERINCLUDES_H
>> > +#endif // LLVM_CLANG_TOOLING_INCLUSIONS_HEADERINCLUDES_H
>> > 
>> > Copied: cfe/trunk/include/clang/Tooling/Inclusions/IncludeStyle.h (from 
>> > r332717, cfe/trunk/include/clang/Tooling/Core/IncludeStyle.h)
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Tooling/Inclusions/IncludeStyle.h?p2=cfe/trunk/include/clang/Tooling/Inclusions/IncludeStyle.h&p1=cfe/trunk/include/clang/Tooling/Core/IncludeStyle.h&r1=332717&r2=332720&rev=332720&view=diff
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Tooling/Inclusions/IncludeStyle.h?p2=cfe/trunk/include/clang/Tooling/Inclusions/IncludeStyle.h&p1=cfe/trunk/include/clang/Tooling/Core/IncludeStyle.h&r1=332717&r2=332720&rev=332720&view=diff>
>> > ==============================================================================
>> > --- cfe/trunk/include/clang/Tooling/Core/IncludeStyle.h (original)
>> > +++ cfe/trunk/include/clang/Tooling/Inclusions/IncludeStyle.h Fri May 18 
>> > 07:16:37 2018
>> > @@ -7,8 +7,8 @@
>> > //
>> > //===----------------------------------------------------------------------===//
>> > 
>> > -#ifndef LLVM_CLANG_TOOLING_CORE_INCLUDESTYLE_H
>> > -#define LLVM_CLANG_TOOLING_CORE_INCLUDESTYLE_H
>> > +#ifndef LLVM_CLANG_TOOLING_INCLUSIONS_INCLUDESTYLE_H
>> > +#define LLVM_CLANG_TOOLING_INCLUSIONS_INCLUDESTYLE_H
>> > 
>> > #include "llvm/Support/YAMLTraits.h"
>> > #include <string>
>> > @@ -130,4 +130,4 @@ struct ScalarEnumerationTraits<
>> > } // namespace yaml
>> > } // namespace llvm
>> > 
>> > -#endif // LLVM_CLANG_TOOLING_CORE_INCLUDESTYLE_H
>> > +#endif // LLVM_CLANG_TOOLING_INCLUSIONS_INCLUDESTYLE_H
>> > 
>> > Modified: cfe/trunk/lib/Format/CMakeLists.txt
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Format/CMakeLists.txt?rev=332720&r1=332719&r2=332720&view=diff
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Format/CMakeLists.txt?rev=332720&r1=332719&r2=332720&view=diff>
>> > ==============================================================================
>> > --- cfe/trunk/lib/Format/CMakeLists.txt (original)
>> > +++ cfe/trunk/lib/Format/CMakeLists.txt Fri May 18 07:16:37 2018
>> > @@ -20,4 +20,5 @@ add_clang_library(clangFormat
>> >   clangBasic
>> >   clangLex
>> >   clangToolingCore
>> > +  clangToolingInclusions
>> >   )
>> > 
>> > Modified: cfe/trunk/lib/Format/Format.cpp
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Format/Format.cpp?rev=332720&r1=332719&r2=332720&view=diff
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Format/Format.cpp?rev=332720&r1=332719&r2=332720&view=diff>
>> > ==============================================================================
>> > --- cfe/trunk/lib/Format/Format.cpp (original)
>> > +++ cfe/trunk/lib/Format/Format.cpp Fri May 18 07:16:37 2018
>> > @@ -31,7 +31,7 @@
>> > #include "clang/Basic/SourceManager.h"
>> > #include "clang/Basic/VirtualFileSystem.h"
>> > #include "clang/Lex/Lexer.h"
>> > -#include "clang/Tooling/Core/HeaderIncludes.h"
>> > +#include "clang/Tooling/Inclusions/HeaderIncludes.h"
>> > #include "llvm/ADT/STLExtras.h"
>> > #include "llvm/ADT/StringRef.h"
>> > #include "llvm/Support/Allocator.h"
>> > 
>> > Modified: cfe/trunk/lib/Tooling/CMakeLists.txt
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/CMakeLists.txt?rev=332720&r1=332719&r2=332720&view=diff
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/CMakeLists.txt?rev=332720&r1=332719&r2=332720&view=diff>
>> > ==============================================================================
>> > --- cfe/trunk/lib/Tooling/CMakeLists.txt (original)
>> > +++ cfe/trunk/lib/Tooling/CMakeLists.txt Fri May 18 07:16:37 2018
>> > @@ -4,6 +4,7 @@ set(LLVM_LINK_COMPONENTS
>> >   )
>> > 
>> > add_subdirectory(Core)
>> > +add_subdirectory(Inclusions)
>> > add_subdirectory(Refactoring)
>> > add_subdirectory(ASTDiff)
>> > 
>> > 
>> > Modified: cfe/trunk/lib/Tooling/Core/CMakeLists.txt
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/Core/CMakeLists.txt?rev=332720&r1=332719&r2=332720&view=diff
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/Core/CMakeLists.txt?rev=332720&r1=332719&r2=332720&view=diff>
>> > ==============================================================================
>> > --- cfe/trunk/lib/Tooling/Core/CMakeLists.txt (original)
>> > +++ cfe/trunk/lib/Tooling/Core/CMakeLists.txt Fri May 18 07:16:37 2018
>> > @@ -2,8 +2,6 @@ set(LLVM_LINK_COMPONENTS support)
>> > 
>> > add_clang_library(clangToolingCore
>> >   Diagnostic.cpp
>> > -  HeaderIncludes.cpp
>> > -  IncludeStyle.cpp
>> >   Lookup.cpp
>> >   Replacement.cpp
>> > 
>> > 
>> > Removed: cfe/trunk/lib/Tooling/Core/HeaderIncludes.cpp
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/Core/HeaderIncludes.cpp?rev=332719&view=auto
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/Core/HeaderIncludes.cpp?rev=332719&view=auto>
>> > ==============================================================================
>> > --- cfe/trunk/lib/Tooling/Core/HeaderIncludes.cpp (original)
>> > +++ cfe/trunk/lib/Tooling/Core/HeaderIncludes.cpp (removed)
>> > @@ -1,330 +0,0 @@
>> > -//===--- HeaderIncludes.cpp - Insert/Delete #includes --*- C++ 
>> > -*----------===//
>> > -//
>> > -//                     The LLVM Compiler Infrastructure
>> > -//
>> > -// This file is distributed under the University of Illinois Open Source
>> > -// License. See LICENSE.TXT for details.
>> > -//
>> > -//===----------------------------------------------------------------------===//
>> > -
>> > -#include "clang/Tooling/Core/HeaderIncludes.h"
>> > -#include "clang/Basic/SourceManager.h"
>> > -#include "clang/Lex/Lexer.h"
>> > -
>> > -namespace clang {
>> > -namespace tooling {
>> > -namespace {
>> > -
>> > -LangOptions createLangOpts() {
>> > -  LangOptions LangOpts;
>> > -  LangOpts.CPlusPlus = 1;
>> > -  LangOpts.CPlusPlus11 = 1;
>> > -  LangOpts.CPlusPlus14 = 1;
>> > -  LangOpts.LineComment = 1;
>> > -  LangOpts.CXXOperatorNames = 1;
>> > -  LangOpts.Bool = 1;
>> > -  LangOpts.ObjC1 = 1;
>> > -  LangOpts.ObjC2 = 1;
>> > -  LangOpts.MicrosoftExt = 1;    // To get kw___try, kw___finally.
>> > -  LangOpts.DeclSpecKeyword = 1; // To get __declspec.
>> > -  LangOpts.WChar = 1;           // To get wchar_t
>> > -  return LangOpts;
>> > -}
>> > -
>> > -// Returns the offset after skipping a sequence of tokens, matched by \p
>> > -// GetOffsetAfterSequence, from the start of the code.
>> > -// \p GetOffsetAfterSequence should be a function that matches a sequence 
>> > of
>> > -// tokens and returns an offset after the sequence.
>> > -unsigned getOffsetAfterTokenSequence(
>> > -    StringRef FileName, StringRef Code, const IncludeStyle &Style,
>> > -    llvm::function_ref<unsigned(const SourceManager &, Lexer &, Token &)>
>> > -        GetOffsetAfterSequence) {
>> > -  SourceManagerForFile VirtualSM(FileName, Code);
>> > -  SourceManager &SM = VirtualSM.get();
>> > -  Lexer Lex(SM.getMainFileID(), SM.getBuffer(SM.getMainFileID()), SM,
>> > -            createLangOpts());
>> > -  Token Tok;
>> > -  // Get the first token.
>> > -  Lex.LexFromRawLexer(Tok);
>> > -  return GetOffsetAfterSequence(SM, Lex, Tok);
>> > -}
>> > -
>> > -// Check if a sequence of tokens is like "#<Name> <raw_identifier>". If 
>> > it is,
>> > -// \p Tok will be the token after this directive; otherwise, it can be 
>> > any token
>> > -// after the given \p Tok (including \p Tok).
>> > -bool checkAndConsumeDirectiveWithName(Lexer &Lex, StringRef Name, Token 
>> > &Tok) {
>> > -  bool Matched = Tok.is <http://tok.is/>(tok::hash) && 
>> > !Lex.LexFromRawLexer(Tok) &&
>> > -                 Tok.is <http://tok.is/>(tok::raw_identifier) &&
>> > -                 Tok.getRawIdentifier() == Name && 
>> > !Lex.LexFromRawLexer(Tok) &&
>> > -                 Tok.is <http://tok.is/>(tok::raw_identifier);
>> > -  if (Matched)
>> > -    Lex.LexFromRawLexer(Tok);
>> > -  return Matched;
>> > -}
>> > -
>> > -void skipComments(Lexer &Lex, Token &Tok) {
>> > -  while (Tok.is <http://tok.is/>(tok::comment))
>> > -    if (Lex.LexFromRawLexer(Tok))
>> > -      return;
>> > -}
>> > -
>> > -// Returns the offset after header guard directives and any comments
>> > -// before/after header guards. If no header guard presents in the code, 
>> > this
>> > -// will returns the offset after skipping all comments from the start of 
>> > the
>> > -// code.
>> > -unsigned getOffsetAfterHeaderGuardsAndComments(StringRef FileName,
>> > -                                               StringRef Code,
>> > -                                               const IncludeStyle &Style) 
>> > {
>> > -  return getOffsetAfterTokenSequence(
>> > -      FileName, Code, Style,
>> > -      [](const SourceManager &SM, Lexer &Lex, Token Tok) {
>> > -        skipComments(Lex, Tok);
>> > -        unsigned InitialOffset = SM.getFileOffset(Tok.getLocation());
>> > -        if (checkAndConsumeDirectiveWithName(Lex, "ifndef", Tok)) {
>> > -          skipComments(Lex, Tok);
>> > -          if (checkAndConsumeDirectiveWithName(Lex, "define", Tok))
>> > -            return SM.getFileOffset(Tok.getLocation());
>> > -        }
>> > -        return InitialOffset;
>> > -      });
>> > -}
>> > -
>> > -// Check if a sequence of tokens is like
>> > -//    "#include ("header.h" | <header.h>)".
>> > -// If it is, \p Tok will be the token after this directive; otherwise, it 
>> > can be
>> > -// any token after the given \p Tok (including \p Tok).
>> > -bool checkAndConsumeInclusiveDirective(Lexer &Lex, Token &Tok) {
>> > -  auto Matched = [&]() {
>> > -    Lex.LexFromRawLexer(Tok);
>> > -    return true;
>> > -  };
>> > -  if (Tok.is <http://tok.is/>(tok::hash) && !Lex.LexFromRawLexer(Tok) &&
>> > -      Tok.is <http://tok.is/>(tok::raw_identifier) && 
>> > Tok.getRawIdentifier() == "include") {
>> > -    if (Lex.LexFromRawLexer(Tok))
>> > -      return false;
>> > -    if (Tok.is <http://tok.is/>(tok::string_literal))
>> > -      return Matched();
>> > -    if (Tok.is <http://tok.is/>(tok::less)) {
>> > -      while (!Lex.LexFromRawLexer(Tok) && Tok.isNot(tok::greater)) {
>> > -      }
>> > -      if (Tok.is <http://tok.is/>(tok::greater))
>> > -        return Matched();
>> > -    }
>> > -  }
>> > -  return false;
>> > -}
>> > -
>> > -// Returns the offset of the last #include directive after which a new
>> > -// #include can be inserted. This ignores #include's after the #include 
>> > block(s)
>> > -// in the beginning of a file to avoid inserting headers into code 
>> > sections
>> > -// where new #include's should not be added by default.
>> > -// These code sections include:
>> > -//      - raw string literals (containing #include).
>> > -//      - #if blocks.
>> > -//      - Special #include's among declarations (e.g. functions).
>> > -//
>> > -// If no #include after which a new #include can be inserted, this 
>> > returns the
>> > -// offset after skipping all comments from the start of the code.
>> > -// Inserting after an #include is not allowed if it comes after code that 
>> > is not
>> > -// #include (e.g. pre-processing directive that is not #include, 
>> > declarations).
>> > -unsigned getMaxHeaderInsertionOffset(StringRef FileName, StringRef Code,
>> > -                                     const IncludeStyle &Style) {
>> > -  return getOffsetAfterTokenSequence(
>> > -      FileName, Code, Style,
>> > -      [](const SourceManager &SM, Lexer &Lex, Token Tok) {
>> > -        skipComments(Lex, Tok);
>> > -        unsigned MaxOffset = SM.getFileOffset(Tok.getLocation());
>> > -        while (checkAndConsumeInclusiveDirective(Lex, Tok))
>> > -          MaxOffset = SM.getFileOffset(Tok.getLocation());
>> > -        return MaxOffset;
>> > -      });
>> > -}
>> > -
>> > -inline StringRef trimInclude(StringRef IncludeName) {
>> > -  return IncludeName.trim("\"<>");
>> > -}
>> > -
>> > -const char IncludeRegexPattern[] =
>> > -    R"(^[\t\ ]*#[\t\ ]*(import|include)[^"<]*(["<][^">]*[">]))";
>> > -
>> > -} // anonymous namespace
>> > -
>> > -IncludeCategoryManager::IncludeCategoryManager(const IncludeStyle &Style,
>> > -                                               StringRef FileName)
>> > -    : Style(Style), FileName(FileName) {
>> > -  FileStem = llvm::sys::path::stem(FileName);
>> > -  for (const auto &Category : Style.IncludeCategories)
>> > -    CategoryRegexs.emplace_back(Category.Regex, llvm::Regex::IgnoreCase);
>> > -  IsMainFile = FileName.endswith(".c") || FileName.endswith(".cc") ||
>> > -               FileName.endswith(".cpp") || FileName.endswith(".c++") ||
>> > -               FileName.endswith(".cxx") || FileName.endswith(".m") ||
>> > -               FileName.endswith(".mm");
>> > -}
>> > -
>> > -int IncludeCategoryManager::getIncludePriority(StringRef IncludeName,
>> > -                                               bool CheckMainHeader) 
>> > const {
>> > -  int Ret = INT_MAX;
>> > -  for (unsigned i = 0, e = CategoryRegexs.size(); i != e; ++i)
>> > -    if (CategoryRegexs[i].match(IncludeName)) {
>> > -      Ret = Style.IncludeCategories[i].Priority;
>> > -      break;
>> > -    }
>> > -  if (CheckMainHeader && IsMainFile && Ret > 0 && 
>> > isMainHeader(IncludeName))
>> > -    Ret = 0;
>> > -  return Ret;
>> > -}
>> > -
>> > -bool IncludeCategoryManager::isMainHeader(StringRef IncludeName) const {
>> > -  if (!IncludeName.startswith("\""))
>> > -    return false;
>> > -  StringRef HeaderStem =
>> > -      llvm::sys::path::stem(IncludeName.drop_front(1).drop_back(1));
>> > -  if (FileStem.startswith(HeaderStem) ||
>> > -      FileStem.startswith_lower(HeaderStem)) {
>> > -    llvm::Regex MainIncludeRegex((HeaderStem + 
>> > Style.IncludeIsMainRegex).str(),
>> > -                                 llvm::Regex::IgnoreCase);
>> > -    if (MainIncludeRegex.match(FileStem))
>> > -      return true;
>> > -  }
>> > -  return false;
>> > -}
>> > -
>> > -HeaderIncludes::HeaderIncludes(StringRef FileName, StringRef Code,
>> > -                               const IncludeStyle &Style)
>> > -    : FileName(FileName), Code(Code), FirstIncludeOffset(-1),
>> > -      MinInsertOffset(
>> > -          getOffsetAfterHeaderGuardsAndComments(FileName, Code, Style)),
>> > -      MaxInsertOffset(MinInsertOffset +
>> > -                      getMaxHeaderInsertionOffset(
>> > -                          FileName, Code.drop_front(MinInsertOffset), 
>> > Style)),
>> > -      Categories(Style, FileName),
>> > -      IncludeRegex(llvm::Regex(IncludeRegexPattern)) {
>> > -  // Add 0 for main header and INT_MAX for headers that are not in any
>> > -  // category.
>> > -  Priorities = {0, INT_MAX};
>> > -  for (const auto &Category : Style.IncludeCategories)
>> > -    Priorities.insert(Category.Priority);
>> > -  SmallVector<StringRef, 32> Lines;
>> > -  Code.drop_front(MinInsertOffset).split(Lines, "\n");
>> > -
>> > -  unsigned Offset = MinInsertOffset;
>> > -  unsigned NextLineOffset;
>> > -  SmallVector<StringRef, 4> Matches;
>> > -  for (auto Line : Lines) {
>> > -    NextLineOffset = std::min(Code.size(), Offset + Line.size() + 1);
>> > -    if (IncludeRegex.match(Line, &Matches)) {
>> > -      // If this is the last line without trailing newline, we need to 
>> > make
>> > -      // sure we don't delete across the file boundary.
>> > -      addExistingInclude(
>> > -          Include(Matches[2],
>> > -                  tooling::Range(
>> > -                      Offset, std::min(Line.size() + 1, Code.size() - 
>> > Offset))),
>> > -          NextLineOffset);
>> > -    }
>> > -    Offset = NextLineOffset;
>> > -  }
>> > -
>> > -  // Populate CategoryEndOfssets:
>> > -  // - Ensure that CategoryEndOffset[Highest] is always populated.
>> > -  // - If CategoryEndOffset[Priority] isn't set, use the next higher value
>> > -  // that is set, up to CategoryEndOffset[Highest].
>> > -  auto Highest = Priorities.begin();
>> > -  if (CategoryEndOffsets.find(*Highest) == CategoryEndOffsets.end()) {
>> > -    if (FirstIncludeOffset >= 0)
>> > -      CategoryEndOffsets[*Highest] = FirstIncludeOffset;
>> > -    else
>> > -      CategoryEndOffsets[*Highest] = MinInsertOffset;
>> > -  }
>> > -  // By this point, CategoryEndOffset[Highest] is always set 
>> > appropriately:
>> > -  //  - to an appropriate location before/after existing #includes, or
>> > -  //  - to right after the header guard, or
>> > -  //  - to the beginning of the file.
>> > -  for (auto I = ++Priorities.begin(), E = Priorities.end(); I != E; ++I)
>> > -    if (CategoryEndOffsets.find(*I) == CategoryEndOffsets.end())
>> > -      CategoryEndOffsets[*I] = CategoryEndOffsets[*std::prev(I)];
>> > -}
>> > -
>> > -// \p Offset: the start of the line following this include directive.
>> > -void HeaderIncludes::addExistingInclude(Include IncludeToAdd,
>> > -                                        unsigned NextLineOffset) {
>> > -  auto Iter =
>> > -      ExistingIncludes.try_emplace(trimInclude(IncludeToAdd.Name)).first;
>> > -  Iter->second.push_back(std::move(IncludeToAdd));
>> > -  auto &CurInclude = Iter->second.back();
>> > -  // The header name with quotes or angle brackets.
>> > -  // Only record the offset of current #include if we can insert after it.
>> > -  if (CurInclude.R.getOffset() <= MaxInsertOffset) {
>> > -    int Priority = Categories.getIncludePriority(
>> > -        CurInclude.Name, /*CheckMainHeader=*/FirstIncludeOffset < 0);
>> > -    CategoryEndOffsets[Priority] = NextLineOffset;
>> > -    IncludesByPriority[Priority].push_back(&CurInclude);
>> > -    if (FirstIncludeOffset < 0)
>> > -      FirstIncludeOffset = CurInclude.R.getOffset();
>> > -  }
>> > -}
>> > -
>> > -llvm::Optional<tooling::Replacement>
>> > -HeaderIncludes::insert(llvm::StringRef IncludeName, bool IsAngled) const {
>> > -  assert(IncludeName == trimInclude(IncludeName));
>> > -  // If a <header> ("header") already exists in code, "header" (<header>) 
>> > with
>> > -  // different quotation will still be inserted.
>> > -  // FIXME: figure out if this is the best behavior.
>> > -  auto It = ExistingIncludes.find(IncludeName);
>> > -  if (It != ExistingIncludes.end())
>> > -    for (const auto &Inc : It->second)
>> > -      if ((IsAngled && StringRef(Inc.Name).startswith("<")) ||
>> > -          (!IsAngled && StringRef(Inc.Name).startswith("\"")))
>> > -        return llvm::None;
>> > -  std::string Quoted = IsAngled ? ("<" + IncludeName + ">").str()
>> > -                                : ("\"" + IncludeName + "\"").str();
>> > -  StringRef QuotedName = Quoted;
>> > -  int Priority = Categories.getIncludePriority(
>> > -      QuotedName, /*CheckMainHeader=*/FirstIncludeOffset < 0);
>> > -  auto CatOffset = CategoryEndOffsets.find(Priority);
>> > -  assert(CatOffset != CategoryEndOffsets.end());
>> > -  unsigned InsertOffset = CatOffset->second; // Fall back offset
>> > -  auto Iter = IncludesByPriority.find(Priority);
>> > -  if (Iter != IncludesByPriority.end()) {
>> > -    for (const auto *Inc : Iter->second) {
>> > -      if (QuotedName < Inc->Name) {
>> > -        InsertOffset = Inc->R.getOffset();
>> > -        break;
>> > -      }
>> > -    }
>> > -  }
>> > -  assert(InsertOffset <= Code.size());
>> > -  std::string NewInclude = ("#include " + QuotedName + "\n").str();
>> > -  // When inserting headers at end of the code, also append '\n' to the 
>> > code
>> > -  // if it does not end with '\n'.
>> > -  // FIXME: when inserting multiple #includes at the end of code, only one
>> > -  // newline should be added.
>> > -  if (InsertOffset == Code.size() && (!Code.empty() && Code.back() != 
>> > '\n'))
>> > -    NewInclude = "\n" + NewInclude;
>> > -  return tooling::Replacement(FileName, InsertOffset, 0, NewInclude);
>> > -}
>> > -
>> > -tooling::Replacements HeaderIncludes::remove(llvm::StringRef IncludeName,
>> > -                                             bool IsAngled) const {
>> > -  assert(IncludeName == trimInclude(IncludeName));
>> > -  tooling::Replacements Result;
>> > -  auto Iter = ExistingIncludes.find(IncludeName);
>> > -  if (Iter == ExistingIncludes.end())
>> > -    return Result;
>> > -  for (const auto &Inc : Iter->second) {
>> > -    if ((IsAngled && StringRef(Inc.Name).startswith("\"")) ||
>> > -        (!IsAngled && StringRef(Inc.Name).startswith("<")))
>> > -      continue;
>> > -    llvm::Error Err = Result.add(tooling::Replacement(
>> > -        FileName, Inc.R.getOffset(), Inc.R.getLength(), ""));
>> > -    if (Err) {
>> > -      auto ErrMsg = "Unexpected conflicts in #include deletions: " +
>> > -                    llvm::toString(std::move(Err));
>> > -      llvm_unreachable(ErrMsg.c_str());
>> > -    }
>> > -  }
>> > -  return Result;
>> > -}
>> > -
>> > -
>> > -} // namespace tooling
>> > -} // namespace clang
>> > 
>> > Removed: cfe/trunk/lib/Tooling/Core/IncludeStyle.cpp
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/Core/IncludeStyle.cpp?rev=332719&view=auto
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/Core/IncludeStyle.cpp?rev=332719&view=auto>
>> > ==============================================================================
>> > --- cfe/trunk/lib/Tooling/Core/IncludeStyle.cpp (original)
>> > +++ cfe/trunk/lib/Tooling/Core/IncludeStyle.cpp (removed)
>> > @@ -1,32 +0,0 @@
>> > -//===--- IncludeStyle.cpp - Style of C++ #include directives -----*- 
>> > C++-*-===//
>> > -//
>> > -//                     The LLVM Compiler Infrastructure
>> > -//
>> > -// This file is distributed under the University of Illinois Open Source
>> > -// License. See LICENSE.TXT for details.
>> > -//
>> > -//===----------------------------------------------------------------------===//
>> > -
>> > -#include "clang/Tooling/Core/IncludeStyle.h"
>> > -
>> > -
>> > -using clang::tooling::IncludeStyle;
>> > -
>> > -namespace llvm {
>> > -namespace yaml {
>> > -
>> > -void MappingTraits<IncludeStyle::IncludeCategory>::mapping(
>> > -    IO &IO, IncludeStyle::IncludeCategory &Category) {
>> > -  IO.mapOptional("Regex", Category.Regex);
>> > -  IO.mapOptional("Priority", Category.Priority);
>> > -}
>> > -
>> > -void 
>> > ScalarEnumerationTraits<IncludeStyle::IncludeBlocksStyle>::enumeration(
>> > -    IO &IO, IncludeStyle::IncludeBlocksStyle &Value) {
>> > -  IO.enumCase(Value, "Preserve", IncludeStyle::IBS_Preserve);
>> > -  IO.enumCase(Value, "Merge", IncludeStyle::IBS_Merge);
>> > -  IO.enumCase(Value, "Regroup", IncludeStyle::IBS_Regroup);
>> > -}
>> > -
>> > -} // namespace yaml
>> > -} // namespace llvm
>> > 
>> > Copied: cfe/trunk/lib/Tooling/Inclusions/CMakeLists.txt (from r332717, 
>> > cfe/trunk/lib/Tooling/Core/CMakeLists.txt)
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/Inclusions/CMakeLists.txt?p2=cfe/trunk/lib/Tooling/Inclusions/CMakeLists.txt&p1=cfe/trunk/lib/Tooling/Core/CMakeLists.txt&r1=332717&r2=332720&rev=332720&view=diff
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/Inclusions/CMakeLists.txt?p2=cfe/trunk/lib/Tooling/Inclusions/CMakeLists.txt&p1=cfe/trunk/lib/Tooling/Core/CMakeLists.txt&r1=332717&r2=332720&rev=332720&view=diff>
>> > ==============================================================================
>> > --- cfe/trunk/lib/Tooling/Core/CMakeLists.txt (original)
>> > +++ cfe/trunk/lib/Tooling/Inclusions/CMakeLists.txt Fri May 18 07:16:37 
>> > 2018
>> > @@ -1,15 +1,12 @@
>> > set(LLVM_LINK_COMPONENTS support)
>> > 
>> > -add_clang_library(clangToolingCore
>> > -  Diagnostic.cpp
>> > +add_clang_library(clangToolingInclusions
>> >   HeaderIncludes.cpp
>> >   IncludeStyle.cpp
>> > -  Lookup.cpp
>> > -  Replacement.cpp
>> > 
>> >   LINK_LIBS
>> > -  clangAST
>> >   clangBasic
>> >   clangLex
>> >   clangRewrite
>> > +  clangToolingCore
>> >   )
>> > 
>> > Copied: cfe/trunk/lib/Tooling/Inclusions/HeaderIncludes.cpp (from r332717, 
>> > cfe/trunk/lib/Tooling/Core/HeaderIncludes.cpp)
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/Inclusions/HeaderIncludes.cpp?p2=cfe/trunk/lib/Tooling/Inclusions/HeaderIncludes.cpp&p1=cfe/trunk/lib/Tooling/Core/HeaderIncludes.cpp&r1=332717&r2=332720&rev=332720&view=diff
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/Inclusions/HeaderIncludes.cpp?p2=cfe/trunk/lib/Tooling/Inclusions/HeaderIncludes.cpp&p1=cfe/trunk/lib/Tooling/Core/HeaderIncludes.cpp&r1=332717&r2=332720&rev=332720&view=diff>
>> > ==============================================================================
>> > --- cfe/trunk/lib/Tooling/Core/HeaderIncludes.cpp (original)
>> > +++ cfe/trunk/lib/Tooling/Inclusions/HeaderIncludes.cpp Fri May 18 
>> > 07:16:37 2018
>> > @@ -7,7 +7,7 @@
>> > //
>> > //===----------------------------------------------------------------------===//
>> > 
>> > -#include "clang/Tooling/Core/HeaderIncludes.h"
>> > +#include "clang/Tooling/Inclusions/HeaderIncludes.h"
>> > #include "clang/Basic/SourceManager.h"
>> > #include "clang/Lex/Lexer.h"
>> > 
>> > 
>> > Copied: cfe/trunk/lib/Tooling/Inclusions/IncludeStyle.cpp (from r332717, 
>> > cfe/trunk/lib/Tooling/Core/IncludeStyle.cpp)
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/Inclusions/IncludeStyle.cpp?p2=cfe/trunk/lib/Tooling/Inclusions/IncludeStyle.cpp&p1=cfe/trunk/lib/Tooling/Core/IncludeStyle.cpp&r1=332717&r2=332720&rev=332720&view=diff
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/Inclusions/IncludeStyle.cpp?p2=cfe/trunk/lib/Tooling/Inclusions/IncludeStyle.cpp&p1=cfe/trunk/lib/Tooling/Core/IncludeStyle.cpp&r1=332717&r2=332720&rev=332720&view=diff>
>> > ==============================================================================
>> > --- cfe/trunk/lib/Tooling/Core/IncludeStyle.cpp (original)
>> > +++ cfe/trunk/lib/Tooling/Inclusions/IncludeStyle.cpp Fri May 18 07:16:37 
>> > 2018
>> > @@ -7,8 +7,7 @@
>> > //
>> > //===----------------------------------------------------------------------===//
>> > 
>> > -#include "clang/Tooling/Core/IncludeStyle.h"
>> > -
>> > +#include "clang/Tooling/Inclusions/IncludeStyle.h"
>> > 
>> > using clang::tooling::IncludeStyle;
>> > 
>> > 
>> > Modified: cfe/trunk/unittests/Tooling/HeaderIncludesTest.cpp
>> > URL: 
>> > http://llvm.org/viewvc/llvm-project/cfe/trunk/unittests/Tooling/HeaderIncludesTest.cpp?rev=332720&r1=332719&r2=332720&view=diff
>> >  
>> > <http://llvm.org/viewvc/llvm-project/cfe/trunk/unittests/Tooling/HeaderIncludesTest.cpp?rev=332720&r1=332719&r2=332720&view=diff>
>> > ==============================================================================
>> > --- cfe/trunk/unittests/Tooling/HeaderIncludesTest.cpp (original)
>> > +++ cfe/trunk/unittests/Tooling/HeaderIncludesTest.cpp Fri May 18 07:16:37 
>> > 2018
>> > @@ -7,7 +7,7 @@
>> > //
>> > //===----------------------------------------------------------------------===//
>> > 
>> > -#include "clang/Tooling/Core/HeaderIncludes.h"
>> > +#include "clang/Tooling/Inclusions/HeaderIncludes.h"
>> > #include "../Tooling/ReplacementTest.h"
>> > #include "../Tooling/RewriterTestContext.h"
>> > #include "clang/Format/Format.h"
>> > 
>> > 
>> > _______________________________________________
>> > cfe-commits mailing list
>> > cfe-commits@lists.llvm.org <mailto:cfe-commits@lists.llvm.org>
>> > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits 
>> > <http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits>
>> 
> 

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to