[PATCH] D57450: [RISCV] Set MaxAtomicInlineWidth and MaxAtomicPromoteWidth for RV32/RV64 targets with atomics

James Clarke via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 19 04:31:20 PST 2019


jrtc27 added a comment.

Also worth noting that the frontend needs to know the max width otherwise `warn_atomic_op_misaligned` will be triggered with the message `large atomic operation may incur significant performance penalty` (name is misleading since the `%select{large|misaligned}0` means it's triggered for both large and misaligned atomic ops, and it's also in the `atomic-alignment` group). This is exactly what I did in my tree (other than the tests) and it seemed to work correctly for RV64A compiling FreeBSD.


Repository:
  rC Clang

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

https://reviews.llvm.org/D57450





More information about the cfe-commits mailing list