[PATCH] D21840: [Driver][CUDA][OpenMP] Reimplement tool selection in the driver.
Alexey Bataev via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 29 19:57:18 PDT 2016
ABataev added a comment.
General comment: remove '\brief' tags, they are not required anymore, just '\\\' is enough
================
Comment at: include/clang/Driver/Action.h:94
@@ -93,1 +93,3 @@
+ /// \brief Flag that is set to true if this action can be collapsed with
+ /// others actions that depend on it. This is true by default and set to false
----------------
'\brief' tag is not required, remove it.
================
Comment at: lib/Driver/Driver.cpp:2400
@@ +2399,3 @@
+ /// \brief Return true if an assemble action can be collapsed.
+ bool canCollapseAssembleAction() {
+ return TC.useIntegratedAs() && !SaveTemps &&
----------------
bool canCollapseAssembleAction() -> bool canCollapseAssembleAction() const
================
Comment at: lib/Driver/Driver.cpp:2408
@@ +2407,3 @@
+ /// \brief Return true if a preprocessor action can be collapsed.
+ bool canCollapsePreprocessorAction() {
+ return !C.getArgs().hasArg(options::OPT_no_integrated_cpp) &&
----------------
Also can be marked as const
================
Comment at: lib/Driver/Driver.cpp:2416
@@ +2415,3 @@
+ /// be collapsed with it.
+ struct JobActionInfoTy {
+ // \brief The action this info refers to.
----------------
I think this can be marked 'final' and add 'nullptr' as a default initializer for 'JA' field
================
Comment at: lib/Driver/Driver.cpp:2424
@@ +2423,3 @@
+
+ /// \brief Append collapsed offload actions from the give nnumber of elements
+ /// in the action info array.
----------------
Remove '\brief'
================
Comment at: lib/Driver/Driver.cpp:2426
@@ +2425,3 @@
+ /// in the action info array.
+ void appendCollapsedOffloadAction(ActionList &CollapsedOffloadAction,
+ ArrayRef<JobActionInfoTy> &ActionInfo,
----------------
I think this function can be marked as 'static', no?
================
Comment at: lib/Driver/Driver.cpp:2435
@@ +2434,3 @@
+
+ /// \brief Functions that attempt to perform the combining. They detect if
+ /// that is legal, and if so they update the inputs \a Inputs and the offload
----------------
remove '\brief'
http://reviews.llvm.org/D21840
More information about the cfe-commits
mailing list