[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