[PATCH] D124942: [BOLT][TEST] Fix MCPlusBuilder::getAliases with two targets

Vladislav Khmelevsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 4 10:22:58 PDT 2022


yota9 added a comment.

Should not the BC



================
Comment at: bolt/lib/Core/MCPlusBuilder.cpp:446
   // alias (are sub or superregs of itself, including itself).
   static std::vector<BitVector> AliasMap;
   static std::vector<BitVector> SmallerAliasMap;
----------------
Thanks, strange that we didn't come across it before. But maybe it would be better to move these vectors to MCPlusBuilder object?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124942



More information about the llvm-commits mailing list