[PATCH] D96912: [flang][fir][NFC] clang-tidy change. Add include.

Kiran Chandramohan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 17 15:47:31 PST 2021


kiranchandramohan accepted this revision.
kiranchandramohan added a comment.
This revision is now accepted and ready to land.

I guess for small format/clang-tidy changes reviews are not required.

You may have a typo in this patch. You can submit with changes (if any needed).

LGTM.



================
Comment at: flang/include/flang/Optimizer/Dialect/FIROps.h:16
 #include "mlir/Interfaces/SideEffectInterfaces.h"
+#include "flang/Optimizer/Dialect/FIRType.h"
+
----------------
Is there a typo here or in line 12? Both refer to the same include file.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96912/new/

https://reviews.llvm.org/D96912



More information about the llvm-commits mailing list