[PATCH] D53491: [Sparc] Add membar assembler tags

James Y Knight via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 4 15:28:26 PST 2018


jyknight added a comment.

Probably worth adding a range check on the membar operand when it's specified as an integer that it's actually in the range 0 <= val <= 127.

And perhaps something on the printing side too. GNU as just reports it as an unknown instruction in that case, but printing the raw integer would be fine too I think.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D53491





More information about the llvm-commits mailing list