[PATCH] D90304: [GlobalISel] Introduce optimal variant of regbankselect

[Dominik Montada via Phabricator via llvm-commits]

gargaroff added inline comments.


================
Comment at: llvm/lib/CodeGen/GlobalISel/RegBankSelect.cpp:1328
+  if (DestReg) {
+    const RegisterBank *RB = MRI->getRegBankOrNull(DestReg);
+    if (RB) {
----------------
This line is causing an assertion with some of our tests .ll tests as it tries to access the regbank of a physical register.

You can provoke the assertion in existing AArch64 tests by adding `RegBankSelect::Optimal` to the RegBankSelect constructor call in `AArch64TargetMachine.cpp`.

One of the tests triggering this for example is `llvm/test/CodeGen/AArch64/GlobalISel/regbank-ceil.ll`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90304