[PATCH] D29441: [Assembler] Enable nicer diagnostics for inline assembly.
Renato Golin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 2 09:34:07 PST 2017
rengolin added inline comments.
================
Comment at: include/llvm/CodeGen/AsmPrinter.h:114
+ /// A SourceMgr for inline assembly.
+ mutable std::unique_ptr<SourceMgr> InlineSrcMgr;
+
----------------
rnk wrote:
> rengolin wrote:
> > I'd move this line down, as it's being handled by the new methods there.
> Most classes seem to group fields together separately from methods, so I think this is fine.
Ok for me, then maybe bring `DiagInfos` up here and add a comment?
https://reviews.llvm.org/D29441
More information about the llvm-commits
mailing list