HsiangKai marked 3 inline comments as done. HsiangKai added inline comments.
================ Comment at: llvm/include/llvm/Support/ARMBuildAttributes.h:37 File = 1, CPU_raw_name = 4, CPU_name = 5, ---------------- jhenderson wrote: > Same comment as elsewhere. I prefer to keep the formatting in this file. We could create another NFC patch for it. ================ Comment at: llvm/lib/Support/ARMBuildAttrs.cpp:15 { ARMBuildAttrs::File, "Tag_File" }, { ARMBuildAttrs::Section, "Tag_Section" }, { ARMBuildAttrs::Symbol, "Tag_Symbol" }, ---------------- jhenderson wrote: > By the way: this clang-format failure might be related to your changes, so > it's probably worth checking to see if it is incorrect without your changes, > and if not, reformat it as part of this patch. I didn't change the formatting in this file. I think it is not related to this patch. We could create another NFC patch to correct the formatting in ARMBuildAttrs.cpp. ================ Comment at: llvm/unittests/Support/ELFAttributeParserTest.cpp:27 +public: + AttributeHeaderParser(ScopedPrinter *sw) + : ELFAttributeParser(sw, emptyTagNameMap, "test") {} ---------------- jhenderson wrote: > `sw` seems a bit of a random name. What does it mean? I do not know either. I borrowed the name from ARMAttributeParser. I will change the variable name to 'printer'. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74023/new/ https://reviews.llvm.org/D74023 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits