[llvm] [DataLayout] Move '*AlignElem' structs and enum inside DataLayout (NFC) (PR #103723)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 14 05:32:15 PDT 2024


================
@@ -159,16 +146,19 @@ class DataLayout {
   /// well-defined bitwise representation.
   SmallVector<unsigned, 8> NonIntegralAddressSpaces;
 
-  /// Attempts to set the alignment of the given type. Returns an error
-  /// description on failure.
-  Error setAlignment(AlignTypeEnum AlignType, Align ABIAlign, Align PrefAlign,
-                     uint32_t BitWidth);
+  // Attempts to set the specification for the given type.
+  // Returns an error description on failure.
----------------
nikic wrote:

It does not matter whether the interface is public or not. `//` would only be appropriate if you want to leave an implementation note (e.g. a TODO).

https://github.com/llvm/llvm-project/pull/103723


More information about the llvm-commits mailing list