[PATCH] D115987: [AVR][MC] Generate section '.progmemX.data' for extended flash banks

Ben Shi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 10 04:42:43 PST 2022


benshi001 marked an inline comment as done.
benshi001 added a comment.

In D115987#3218125 <https://reviews.llvm.org/D115987#3218125>, @aykevl wrote:

> I wonder whether we should generate an error instead of falling back to other memory areas? That would be more consistent with avr-gcc and I think that would also be more correct.

Actually an error will be reported by clang as in https://reviews.llvm.org/D115982, which is indeed consistent with avr-gcc.

And in current patch, a brief error will be reported too (though the location information needs to be filled by  future patches).


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

https://reviews.llvm.org/D115987



More information about the llvm-commits mailing list