[PATCH] D114557: [fir] Add base for runtime builder unittests
Diana Picus via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 25 00:40:56 PST 2021
rovka added inline comments.
================
Comment at: flang/unittests/Optimizer/Builder/Runtime/RuntimeCallTestBase.h:86
+
+static void checkCallOpFromResultBox(mlir::Value result,
+ llvm::StringRef fctName, unsigned nbArgs, bool addLocArgs = true) {
----------------
I think this needs a comment about exactly what it's checking (e.g. some fir snippet), so it's easier for people to know when to use it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114557/new/
https://reviews.llvm.org/D114557
More information about the llvm-commits
mailing list