[PATCH] D50231: [llvm-exegesis] Renaming classes and functions.

[Guillaume Chatelet via Phabricator via llvm-commits]

gchatelet marked an inline comment as done.
gchatelet added inline comments.


================
Comment at: llvm/trunk/unittests/tools/llvm-exegesis/X86/SnippetGeneratorTest.cpp:221
 
-TEST_F(UopsSnippetGeneratorTest, MemoryUse) {
+TEST_F(UopsBenchmarkRunnerTest, MemoryUse) {
   // Mov32rm reads from memory.
----------------
yubing wrote:
> Hi, gchatelet:
>   I just wonder how to run this testcase. Could you please give us some help?
> 
> Thanks
> 
Sure, the target you want to use is `unittests/tools/llvm-exegesis/X86/all`
Here is the full cmd line I'm using
```
> cmake -DCMAKE_BUILD_TYPE=Debug -DLLVM_BUILD_TESTS=ON -DBUILD_SHARED_LIBS=ON -DLLVM_USE_SPLIT_DWARF=ON -DLLVM_OPTIMIZED_TABLEGEN=ON -DLLVM_ENABLE_WERROR=ON -H/redacted/git/llvm-project/llvm -B/tmp/llvm-project_dbg_compiled-with-clang -GNinja
> ninja -C /tmp/llvm-project_dbg_compiled-with-clang unittests/tools/llvm-exegesis/X86/all
> /tmp/llvm-project_dbg_compiled-with-clang/unittests/tools/llvm-exegesis/X86/LLVMExegesisX86Tests
```


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D50231/new/

https://reviews.llvm.org/D50231