address reviews

Author: bcardosolopes

mlir/include/mlir/Interfaces/DataLayoutInterfaces.h

@@ -101,8 +101,8 @@ Attribute getDefaultGlobalMemorySpace(DataLayoutEntryInterface entry);
 /// DataLayoutInterface if specified, otherwise returns the default.
 uint64_t getDefaultStackAlignment(DataLayoutEntryInterface entry);
 
-/// Default handler for the stack alignment request. Dispatches to the
-/// DataLayoutInterface if specified, otherwise returns the default.
+/// Default handler for the function pointer alignment request. Dispatches to
+/// the DataLayoutInterface if specified, otherwise returns the default.
 Attribute getDefaultFunctionPointerAlignment(DataLayoutEntryInterface entry);
 
 /// Returns the value of the property from the specified DataLayoutEntry. If the

mlir/lib/Interfaces/DataLayoutInterfaces.cpp

@@ -316,9 +316,8 @@ mlir::detail::getDefaultStackAlignment(DataLayoutEntryInterface entry) {
 // the entry is empty the default alignment zero is returned.
 Attribute mlir::detail::getDefaultFunctionPointerAlignment(
     DataLayoutEntryInterface entry) {
-  if (entry == DataLayoutEntryInterface()) {
+  if (entry == DataLayoutEntryInterface())
     return Attribute();
-  }
   return entry.getValue();
 }