@revane sure, I'll rename those files files.

================
Comment at: clang-modernize/Core/Transform.cpp:102-105
@@ +101,6 @@
+
+  llvm::SmallString<256> FilePath(Filename);
+  llvm::error_code EC = llvm::sys::fs::make_absolute(FilePath);
+  assert(!EC && "File path can't be made absolute");
+  (void) EC;
+
----------------
Edwin Vane wrote:
> Wouldn't it be better to do this once per source file in ClangModernizer.cpp 
> instead of once per source per transform here?
No problem, but I'm not convinced that this is the right approach, if we move 
the YAML files to a folder this will go away.


http://llvm-reviews.chandlerc.com/D1837
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to