compnerd accepted this revision. compnerd added a comment. Generally, `clang-format` the changes, it will catch the formatting things.
================ Comment at: lldb/source/Plugins/SymbolFile/NativePDB/PdbFPOProgramToDWARFExpression.cpp:28 + llvm::codeview::CPUType cpu; + switch(arch_type) { + case llvm::Triple::ArchType::aarch64: ---------------- Space after the `switch`. ================ Comment at: lldb/source/Plugins/SymbolFile/NativePDB/PdbFPOProgramToDWARFExpression.cpp:35 + default: + cpu = llvm::codeview::CPUType::X64; + } ---------------- Please add a `break` after this. Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62771/new/ https://reviews.llvm.org/D62771 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits