From ea3574561020081d84459a6e8028b59ff3814000 Mon Sep 17 00:00:00 2001 From: Eric Schweitz Date: Fri, 5 Feb 2021 08:53:32 -0800 Subject: [PATCH] [flang][NFC] Update comments. Differential Revision: https://reviews.llvm.org/D96152 --- flang/include/flang/Optimizer/Dialect/FIRType.h | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/flang/include/flang/Optimizer/Dialect/FIRType.h b/flang/include/flang/Optimizer/Dialect/FIRType.h index c6fa318acf78..c574665eaf78 100644 --- a/flang/include/flang/Optimizer/Dialect/FIRType.h +++ b/flang/include/flang/Optimizer/Dialect/FIRType.h @@ -5,6 +5,10 @@ // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception // //===----------------------------------------------------------------------===// +// +// Coding style: https://mlir.llvm.org/getting_started/DeveloperGuide/ +// +//===----------------------------------------------------------------------===// #ifndef OPTIMIZER_DIALECT_FIRTYPE_H #define OPTIMIZER_DIALECT_FIRTYPE_H @@ -67,7 +71,9 @@ bool isa_fir_or_std_type(mlir::Type t); /// Is `t` a FIR dialect type that implies a memory (de)reference? bool isa_ref_type(mlir::Type t); -/// Is `t` a type that is always trivially pass-by-reference? +/// Is `t` a type that is always trivially pass-by-reference? Specifically, this +/// is testing if `t` is a ReferenceType or any box type. Compare this to +/// conformsWithPassByRef(), which includes pointers and allocatables. bool isa_passbyref_type(mlir::Type t); /// Is `t` a boxed type?