<div dir="ltr"><div style><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">Could you give a bit more background on the motivation for this? I wasn't able to find any discussion on llvmdev.</span></div>
<div><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">+// This pass replaces occurences of __nvvm_reflect("string") with an</span><br style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">
<span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">+// integer based on -nvvm-reflect-list string=<int> option given to this pass.</span><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><br>
</span></div><div><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><br></span></div><div style><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">Please give an example of usage here.</span></div>
<div style><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><br></span></div><div style><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">It also seems that in some places you say "0/1", but in this description you say "<int>". Is any int allowed? Please clarify.</span></div>
<div style><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><br></span></div><div style><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">+    int reflectVal = 0; // The default value is 0</span><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><br>
</span></div><div style><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><br></span></div><div style><font color="#000000" face="arial, sans-serif">Please put this information in the top-of-the-file comment.</font></div>
<span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><div><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><br></span></div><div>+  for (unsigned i = 0, e = ReflectList.size(); i != e; ++i) {<br>
+    //    DEBUG(dbgs() << "Option : "  << ReflectList[i] << std::endl);<span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><br></span></div><div><br></div><div>+    reflectArg = reflectArg.substr(0, reflectArg.size() - 1);<br>
+    //    DEBUG(dbgs() << "Arg of _reflect : " << reflectArg << std::endl);<span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><br></span></div><div><br></div>+  //std::map<std::string, int> VarMap;</span><br style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">
<span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">+  StringMap<int> VarMap;</span><br><div><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><br></span></div><div style>
<span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">Please remove commented-out code.</span></div><div style><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><br></span></div>
<div style><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">+static std::vector<std::string></span><br style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">+Tokenize(const std::string &str, const std::string &delim) {</span><br style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">
<span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">+  std::vector<std::string> tokens;</span><br style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px"><br></div><div style><font color="#000000" face="arial, sans-serif">Is there a particular reason you aren't following the LLVM naming convention here and throughout?</font></div>
<div style><font color="#000000" face="arial, sans-serif"><br></font></div><div style><font color="#000000" face="arial, sans-serif">-- Sean Silva</font></div></div>