[PATCH] D91519: [AST][Mach0] Fix unused-variable warnings

2020-11-19 Thread David Stenberg via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG2d1f471e45af: [Mach0] Fix unused-variable warnings (authored by ehjogab, committed by dstenb). Herald added a project: LLVM. Changed prior to

[PATCH] D91519: [AST][Mach0] Fix unused-variable warnings

2020-11-17 Thread Gabriel Hjort Ã…kerlund via Phabricator via cfe-commits
ehjogab updated this revision to Diff 305984. ehjogab added a comment. Replace dyn_cast with isa Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91519/new/ https://reviews.llvm.org/D91519 Files: clang/lib/AST/APValue.cpp

[PATCH] D91519: [AST][Mach0] Fix unused-variable warnings

2020-11-16 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm added inline comments. Comment at: lld/MachO/SymbolTable.cpp:137 // error message. -if (auto *defined = dyn_cast(s)) +if (dyn_cast(s)) error("found defined symbol with illegal name " + DSOHandle::name); Should use isa Repository:

[PATCH] D91519: [AST][Mach0] Fix unused-variable warnings

2020-11-16 Thread Gabriel Hjort Ã…kerlund via Phabricator via cfe-commits
ehjogab created this revision. ehjogab added reviewers: dsanders, arsenm, qcolombet, ab, bjope, rovka. Herald added a project: clang. Herald added a subscriber: cfe-commits. ehjogab requested review of this revision. Herald added a subscriber: wdng. Change-Id: