[PATCH] D154793: [DWARFLinkerParallel] Add support of accelerator tables to DWARFLinkerParallel.

Alexey Lapshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 22 02:40:07 PDT 2023


avl added inline comments.


================
Comment at: llvm/lib/DWARFLinkerParallel/DIEAttributeCloner.h:23-28
+  AttributesInfo() {
+    HasLiveAddress = false;
+    IsDeclaration = false;
+    HasRanges = false;
+    HasStringOffsetBaseAttr = false;
+  }
----------------
JDevlieghere wrote:
> Can we continue to do the initialization inline? 
After C++20 would be supported :-)  AFAIK, The default bitfield member initialization is a C++20 feature.

Probably, we can remove bitfields usages here as in current configuration they do not buy anything.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D154793/new/

https://reviews.llvm.org/D154793



More information about the llvm-commits mailing list