[PATCH] D25278: [AVR] Add AVRISelDAGToDAG.cpp

Krzysztof Parzyszek via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 19 10:56:27 PDT 2016


kparzysz accepted this revision.
kparzysz added a comment.
This revision is now accepted and ready to land.

The check for NON_EXTLOAD seems to be wrong again.  LGTM otherwise.



================
Comment at: lib/Target/AVR/AVRISelDAGToDAG.cpp:129
+  // We only care if this load uses a POSTINC or PREDEC mode.
+  if ((LD->getExtensionType() != ISD::NON_EXTLOAD) ||
+      (AM != ISD::POST_INC && AM != ISD::PRE_DEC)) {
----------------
This has become != again.


https://reviews.llvm.org/D25278





More information about the llvm-commits mailing list