[PATCH] D99631: [RISCV] Refine pre-define macro tests

[Jim Lin via Phabricator via cfe-commits]

Jim created this revision.
Jim added reviewers: craig.topper, simoncook, HsiangKai.
Herald added subscribers: vkmr, frasercrmck, evandro, luismarques, apazos, sameer.abuasal, s.egerton, benna, psnobl, jocewei, PkmX, jfb, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, johnrusso, rbar, asb.
Jim requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

1. Undefined macro test for rv32i and rv64i.
  1. Reorder it with canonical order.
  2. Add missing  undefined macro check.
  3. Append defined value to `__riscv_a`, `__riscv_f` and `__riscv_c` to distinguish with `__riscv_arch_test`, `__riscv_cmodel_medlow` and `__riscv_float_abi_soft`. They have the same prefix.
2. Move abi macro test below f and d.
3. Unify coding style for newline.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D99631

Files:
  clang/test/Preprocessor/riscv-target-features.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D99631.334316.patch
Type: text/x-patch
Size: 14422 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210331/d9588783/attachment.bin>