[PATCH] D50929: [llvm-mca] Introduce the llvm-mca library and organize the directory accordingly. NFC.

Matt Davis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 20 09:44:40 PDT 2018


mattd added a comment.

In https://reviews.llvm.org/D50929#1205986, @andreadb wrote:

> My opinion is that `CodeRegion.h` and `CodeRegion.cpp` should not be part of the library. Those files implement a class which is meant to be used by the llvm-mca driver only to mark regions of code. A different utility class/approach should be used when llvm-mca is used as a library.


I am fine with that, I'll move those utility pieces outside of the library.

>> I would like to place the Views and Stats into separate (non-library) directories, just to keep things better organized, but that would be a separate patch.
> 
> Any reasons why it cannot be done as part of this patch?

I was just trying to isolate this patch to the addition of a library, but I am happy to move those files in this patch too.


https://reviews.llvm.org/D50929





More information about the llvm-commits mailing list