================
Comment at: clang-tidy/llvm/LLVMModule.cpp:88
@@ +87,3 @@
+ StringRef SourceFile = Sources.getFilename(HashLoc);
+ if (!SourceFile.endswith(".cc"))
+ return;
----------------
Sean Silva wrote:
> Daniel Jasper wrote:
> > Sean Silva wrote:
> > > Shouldn't this be checking LangOptions instead of the extension? e.g.
> > > this wouldn't work for LLVM's `.cpp` naming.
> > This is a dummy implementation. Any real order-check will be totally
> > different.
> Please indicate this clearly somewhere!
Well, I intend to clean this up some more before submitting .. I thought this
was clear enough for a first architectural review ..
http://llvm-reviews.chandlerc.com/D884
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
