[PATCH] D153742: [RISCV] Add Zce extension.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

In D153742#4449703 <https://reviews.llvm.org/D153742#4449703>, @jrtc27 wrote:

> What does that mean for `.option arch, +zce, +f` with -march=rv32i? Should that result in Zcf being present or not? My guess is binutils's dodgy implementation (which only considers implications at the end, so `.option arch, foo, bar` is not just shorthand for `.option arch, foo; .option arch, bar` as it should be in a sane world) says yes but we say no.

The easiest fix for that is probably to factor Zce directly into the tablegen predicates?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153742