[PATCH] D30497: [SelectionDAG] Make SelectCode return void

Justin Bogner via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 1 16:26:48 PST 2017


LGTM.

Sven van Haastregt via Phabricator <reviews at reviews.llvm.org> writes:
> svenvh created this revision.
>
> SelectCode has been returning nullptr since 182dac0 ("SDAG: Make
> SelectCodeCommon return void", 2016-05-10).  Make SelectCode also
> return void instead, as all callers have been updated.
>
>
> https://reviews.llvm.org/D30497
>
> Files:
>   utils/TableGen/DAGISelMatcherEmitter.cpp
>
>
> Index: utils/TableGen/DAGISelMatcherEmitter.cpp
> ===================================================================
> --- utils/TableGen/DAGISelMatcherEmitter.cpp
> +++ utils/TableGen/DAGISelMatcherEmitter.cpp
> @@ -955,7 +955,7 @@
>    formatted_raw_ostream OS(O);
>  
>    OS << "// The main instruction selector code.\n";
> -  OS << "SDNode *SelectCode(SDNode *N) {\n";
> +  OS << "void SelectCode(SDNode *N) {\n";
>  
>    MatcherTableEmitter MatcherEmitter(CGP);
>  
> @@ -970,7 +970,6 @@
>  
>    OS << "  #undef TARGET_VAL\n";
>    OS << "  SelectCodeCommon(N, MatcherTable,sizeof(MatcherTable));\n";
> -  OS << "  return nullptr;\n";
>    OS << "}\n";
>  
>    // Next up, emit the function for node and pattern predicates:
>
>
> Index: utils/TableGen/DAGISelMatcherEmitter.cpp
> ===================================================================
> --- utils/TableGen/DAGISelMatcherEmitter.cpp
> +++ utils/TableGen/DAGISelMatcherEmitter.cpp
> @@ -955,7 +955,7 @@
>    formatted_raw_ostream OS(O);
>  
>    OS << "// The main instruction selector code.\n";
> -  OS << "SDNode *SelectCode(SDNode *N) {\n";
> +  OS << "void SelectCode(SDNode *N) {\n";
>  
>    MatcherTableEmitter MatcherEmitter(CGP);
>  
> @@ -970,7 +970,6 @@
>  
>    OS << "  #undef TARGET_VAL\n";
>    OS << "  SelectCodeCommon(N, MatcherTable,sizeof(MatcherTable));\n";
> -  OS << "  return nullptr;\n";
>    OS << "}\n";
>  
>    // Next up, emit the function for node and pattern predicates:


More information about the llvm-commits mailing list