[PATCH] D61130: [llvm-mc] Add reportWarning() to MCContext

sidneym accepted this revision. sidneym added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61130/new/ https://reviews.llvm.org/D61130

[PATCH] D61130: [llvm-mc] Add reportWarning() to MCContext

bcain added a comment. More context for this change: - "`--no-warn`" is present already, target-independent, and it's supported by llvm-mc (and other MC tools) because it's already present in `include/llvm/MC/MCTargetOptionsCommandFlags.inc`. - Only the AsmParser has access to the

[PATCH] D61130: [llvm-mc] Add reportWarning() to MCContext

bcain added a comment. ping! any comments or concerns about this change? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61130/new/ https://reviews.llvm.org/D61130 ___ cfe-commits mailing list

[PATCH] D61130: [llvm-mc] Add reportWarning() to MCContext

bcain updated this revision to Diff 197317. bcain changed the repository for this revision from rL LLVM to rG LLVM Github Monorepo. bcain added a comment. Herald added a project: clang. Herald added a subscriber: cfe-commits. Add context, add clang update. Repository: rG LLVM Github Monorepo