[PATCH] D107528: [llvm][clang][NFC] updates inline licence info
    Aaron Ballman via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Aug  5 05:13:38 PDT 2021
    
    
  
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
There are some spurious whitespace changes that should be dropped, but otherwise, this LGTM. Thank you for the cleanup!
================
Comment at: clang/lib/Tooling/DependencyScanning/ModuleDepCollector.cpp:115
     return;
-  
   // This has to be delayed as the context hash can change at the start of
----------------
Spurious whitespace change.
================
Comment at: llvm/lib/Analysis/DevelopmentModeInlineAdvisor.cpp:35
     cl::desc(R"(Path to SavedModel from the previous training iteration.
-The directory is also expected to contain a JSON specification of the 
-outputs expected to be logged, where the first entry must be the 
-inlining decision. The file containing the specification should be 
-called output_spec.json. The expected JSON value is an array of 
-dictionaries. Each dictionary should have 2 keys: 
+The directory is also expected to contain a JSON specification of the
+outputs expected to be logged, where the first entry must be the
----------------
Unrelated whitespace changes.
================
Comment at: llvm/lib/BinaryFormat/MsgPackDocumentYAML.cpp:249
 }
-
----------------
Unrelated whitespace change.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107528/new/
https://reviews.llvm.org/D107528
    
    
More information about the llvm-commits
mailing list