[PATCH] D36820: [Bash-autocompletion] Add support for -std=

Yuka Takahashi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 17 13:05:34 PDT 2017


yamaguchi added inline comments.


================
Comment at: llvm/utils/TableGen/OptParserEmitter.cpp:314
       for (const std::string &Pref : R.getValueAsListOfStrings("Prefixes")) {
-        OS << "bool ValuesWereAdded = ";
+        OS << "ValuesWereAdded = ";
         OS << "Opt.addValues(";
----------------
ruiu wrote:
> Why do you have to change this? The previous code looks nicer.
Because I got an error that ValuesWereAdded are defined several times.


https://reviews.llvm.org/D36820





More information about the cfe-commits mailing list