[PATCH] D78459: [AVR] Don't adjust for instruction size

Dylan McKay via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 31 04:47:14 PDT 2020


dylanmckay added a comment.

Added an integration test for this and verified that the test does indeed miscompile and crash the AVR simulator before the patch, but work correctly afterwards. The testcase output is also consistent with AVR-GCC.

https://github.com/dylanmckay/avr-compiler-integration-tests/commit/6c4c67787c3525ec719d4834180874966dbfb2af

I will approve this patch once the `avr-jmp.c` test is added to the clang test suite. in this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78459





More information about the llvm-commits mailing list