[PATCH] D36742: [LLD][ELF][AArch64] Implement scanner for Cortex-A53 Erratum 843419

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 29 03:29:35 PDT 2018


grimar added inline comments.
Herald added a reviewer: javed.absar.
Herald added a reviewer: espindola.


================
Comment at: ELF/AArch64ErrataFix.cpp:256
+static bool isV8NonStructureLoad(uint32_t Instr) {
+  if (isLoadExclusive(Instr))
+    return true;
----------------
Hi from 2018, Peter :)

I am sorry for the delay, but this line of the code seems uncovered by any test cases we have.
Do you think you can fix it? (If not - I'll try to do something by myself probably).

(We are somewhere in the middle of the doing the bot to check the LLD code coverage.
I am currently trying to find out and resolve the most problematic places)


Repository:
  rL LLVM

https://reviews.llvm.org/D36742





More information about the llvm-commits mailing list