================
@@ -52,10 +62,33 @@ class CIRGenModule : public CIRGenTypeCache {
   /// A "module" matches a c/cpp source file: containing a list of functions.
   mlir::ModuleOp theModule;
 
+  clang::DiagnosticsEngine &diags;
+
   const clang::TargetInfo ⌖
 
 public:
+  mlir::ModuleOp getModule() const { return theModule; }
+
+  /// Helpers to convert Clang's SourceLocation to an MLIR Location.
----------------
dkolsen-pgi wrote:

I ran some tests.  ClangIR (both the incubator and this PR) uses the presumed 
location from the `SourceLocation` when creating an `mlir::Location`.  It 
honors `#line` directives and records the location where the macro was 
expanded.  That is the location we most want to keep around.

I changed the comment to say that the presumed location is what is converted.


https://github.com/llvm/llvm-project/pull/113483
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to