[Lldb-commits] [PATCH] D85641: [LLDB] Convert SVE macros into c++ constants and inlines

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Aug 11 04:06:16 PDT 2020

labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.

I originally thought we would keep the original macro capitalization (similar to how `llvm/BinaryFormat/elf.h` maintains the elf.h capitalizations even though it does not define these symbols as macros). I still think that would be a good idea, but I don't mind the current version either. If you do keep the current version, then I think we should also change the namespace name to lower case (`sve::`).

Comment at: lldb/source/Plugins/Process/Utility/LinuxPTraceDefines_arm64sve.h:22
- * See linux/Documentation/arm64/sve.txt for a description of the VL/VQ
+ * See Linux kernel Documentation/s/sve.rst for a description of the vl/vq
  * terminology.
too aggressive search&replace ?



More information about the lldb-commits mailing list