[PATCH] D149032: [InlineAsm][AArch64]Add backend support for flag output parameters

[Mingming Liu via Phabricator via llvm-commits]

mingmingl planned changes to this revision.
mingmingl added a comment.

In D149032#4296049 <https://reviews.llvm.org/D149032#4296049>, @efriedma wrote:

> Adding a fallback from globalisel to selectiondag isel should be simple;

Thanks for pointing out fallback mechanism (`TargetLowering::fallBackToDAGISel`)! Will do

> I'd like to avoid advertising support for functionality that doesn't actually work at all optimization levels.

This is a valid point and sounds reasonable.

> This is probably worth noting in release notes.

will do

In D149032#4296049 <https://reviews.llvm.org/D149032#4296049>, @efriedma wrote:

> Adding a fallback from globalisel to selectiondag isel should be simple; I'd like to avoid advertising support for functionality that doesn't actually work at all optimization levels.
>
> This is probably worth noting in release notes.




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

https://reviews.llvm.org/D149032