[PATCH] D108727: [X86][MCA] Address other issues with MULX reported in PR51495.

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 25 14:23:06 PDT 2021


lebedev.ri added a comment.

THis



================
Comment at: llvm/lib/MCA/HardwareUnits/RegisterFile.cpp:293
+    // instruction to register RegID.
+    const WriteRef &OtherWrite = RegISterMappings[RegID].first;
+    const WriteState *OtherWS = OtherWrite.getWriteState();
----------------
This doesn't compile for me
```
/repositories/llvm-project/llvm/lib/MCA/HardwareUnits/RegisterFile.cpp:293:34: error: use of undeclared identifier 'RegISterMappings'; did you mean 'RegisterMappings'?
    const WriteRef &OtherWrite = RegISterMappings[RegID].first;
                                 ^~~~~~~~~~~~~~~~
                                 RegisterMappings
/repositories/llvm-project/llvm/include/llvm/MCA/HardwareUnits/RegisterFile.h:190:32: note: 'RegisterMappings' declared here
  std::vector<RegisterMapping> RegisterMappings;
                               ^

```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108727



More information about the llvm-commits mailing list