Hello, The recent renamings moved some IR-related files and directories to more logical places. Is there a plan to rename lib/AsmParser as well? This directory name is somewhat misleading, because the code in it parses IR. In fact, nothing mentions AsmParser in the code inside that dir, except the build files. The name creates a confusion with other AsmParser citizens of LLVM like lib/MC/MCParser/AsmParser.cpp (which is an actual assembly parser, class AsmParser). Eli
On Jan 16, 2013, at 10:42 AM, Eli Bendersky <eliben at google.com> wrote:> The recent renamings moved some IR-related files and directories to > more logical places. Is there a plan to rename lib/AsmParser as well? > This directory name is somewhat misleading, because the code in it > parses IR. In fact, nothing mentions AsmParser in the code inside that > dir, except the build files. The name creates a confusion with other > AsmParser citizens of LLVM like lib/MC/MCParser/AsmParser.cpp (which > is an actual assembly parser, class AsmParser).Renaming it to lib/IRParser would make sense to me (along with corresponding things in test/ etc). -Chris
Seemingly Similar Threads
- [LLVMdev] bug report on MC asm parser evaluating expressions
- [LLVMdev] bug report on MC asm parser evaluating expressions
- [LLVMdev] RFC: A Great Renaming of Things (or: Let's Repaint ALL the Bikesheds!)
- [LLVMdev] MC: Object file specific parsing
- -Wmisleading-indentation violations