[PATCH] D96340: Make fixed-abi default for AMD HSA OS

Jon Chesterfield via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 11 09:58:10 PST 2021


JonChesterfield added a comment.

Setting this 'true' on everything is expected to be a regression. Suggest reverting this patch.

Alternative would look something like:

- if there is any indirect call, set this true
- otherwise, leave it unchanged

That will fix things using indirect calls without regressing everything else.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96340



More information about the llvm-commits mailing list