[PATCH] D86955: [Hexagon] Make HexagonVLCR compatibile with New PM

Tue Sep 8 11:24:28 PDT 2020

pzheng added a comment.

In D86955#2261510 <https://reviews.llvm.org/D86955#2261510>, @quic_aankit wrote:

> In D86955#2261498 <https://reviews.llvm.org/D86955#2261498>, @pzheng wrote:
>
>> Is there any existing test case for this pass with the legacy pass manager? If so, you might want to add one with the new pass manager too.
>
> All the existing testcases use opt to run the pass. The pass is not run integrated in the main list of passes for any -Ox level.

That's what I meant. For each legacy pass manager test "RUN: opt -hexagon-vlcr ...", you can add a corresponding new pass manager test "RUN: opt -passes=hexagon-vlcr". This way, we will have the same test coverage with both the legacy and new pass managers.

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

https://reviews.llvm.org/D86955