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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits