[PATCH] D122533: [AVR] Remove AVRRelaxMemOperations

Patryk Wychowaniec via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Mar 27 22:51:50 PDT 2022


Patryk27 added a comment.

In D122533#3410428 <https://reviews.llvm.org/D122533#3410428>, @benshi001 wrote:

> In D122533#3410259 <https://reviews.llvm.org/D122533#3410259>, @Patryk27 wrote:
>
>> Ok, I have added the switch; I think a separate patch that adds that switch to all of the AVR tests could come handy - what do you think?
>
> No. `-verify-machineinstrs` might be default in the future, which is still under discuss, since it costs longer time. Currently we just make highly risky case with explicit `-verify-machineinstrs`, such as yours. ^_^

Okie, understood - thanks :-)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122533



More information about the cfe-commits mailing list