[PATCH] D64827: [Xtensa 2/10] Add Xtensa ELF definitions.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 19 16:00:17 PST 2022


MaskRay accepted this revision.
MaskRay added inline comments.


================
Comment at: llvm/unittests/Object/ELFObjectFileTest.cpp:317
+  size_t I = 0;
+  for (const DataForTest &D : generateData(ELF::EM_XTENSA))
+    checkFormatAndArch(D, Formats[I++], Triple::xtensa);
----------------
This can use `llvm::enumerate`, but it's ok to follow other tests.


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

https://reviews.llvm.org/D64827



More information about the llvm-commits mailing list