[llvm] [MachineBlockPlacement][X86] Use max of MDAlign and TLIAlign to align Loops. (PR #71026)

Shengchen Kan via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 20 19:01:24 PST 2023


================
@@ -66,6 +67,14 @@ class MachineLoop : public LoopBase<MachineBasicBlock, MachineLoop> {
   /// it returns an unknown location.
   DebugLoc getStartLoc() const;
 
+  /// \brief Find the llvm.loop metadata for this loop.
+  /// If each branch to the header of this loop contains the same llvm.loop
----------------
KanRobert wrote:

Should we add a blank line after line 70?
https://llvm.org/docs/CodingStandards.html

> The first sentence (or a passage beginning with \brief) is used as an abstract. Any additional information should be separated by a blank line. If an algorithm is based on a paper or is described in another source, provide a reference.

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


More information about the llvm-commits mailing list