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
_
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 `MCTargetOpti
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
cfe-commi
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