[llvm] [CodeGen] Use BasicBlock numbers to map to MBBs (PR #101883)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 5 10:38:07 PDT 2024


================
@@ -73,8 +73,8 @@ class FunctionLoweringInfo {
   /// allocated to hold a pointer to the hidden sret parameter.
   Register DemoteRegister;
 
-  /// MBBMap - A mapping from LLVM basic blocks to their machine code entry.
-  DenseMap<const BasicBlock*, MachineBasicBlock *> MBBMap;
+  /// MBBMap - A mapping from LLVM basic block number to their machine block.
----------------
MaskRay wrote:

https://llvm.org/docs/CodingStandards.html#doxygen-use-in-documentation-comments says "Don’t duplicate function or class name at the beginning of the comment."

As you are updating the comment, drop `MBBMap - `

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


More information about the llvm-commits mailing list