[PATCH] D93054: [ARM] Add bank conflict hazarding

[David Penry via Phabricator via llvm-commits]

dpenry marked 2 inline comments as done.
dpenry added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMHazardRecognizer.cpp:225
+        BasePseudoVal0->kind() == BasePseudoVal1->kind() &&
+        BasePseudoVal0->isConstantPool() && AssumeITCMBankConflict)
+      return Hazard;
----------------
SjoerdMeijer wrote:
> Option `AssumeITCMBankConflict` defaults to false. Do have/need a test with this enabled?
For Cortex-M7, this flag is enabled.  There isn't a test written to distinguish between enabled/disabled at present.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93054