imkiva added a comment.
Looks like I am reading a very old version of the spec haha. Anyway, thank you
all for giving me the newest information and useful links! Thanks!!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156214/new/
https://reviews.ll
imkiva added a comment.
address review comments
Comment at: llvm/lib/Support/RISCVISAInfo.cpp:948
- // TODO: The 'q' extension requires rv64.
- // TODO: It is illegal to specify 'e' extensions with 'f' and 'd'.
wangpc wrote:
> I think the comment is outdate
imkiva updated this revision to Diff 543890.
imkiva added a comment.
Removed some tests that used `e` together with `f` or `d`
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156214/new/
https://reviews.llvm.org/D156214
Files:
clang/test/Driver/ri
imkiva created this revision.
imkiva added a reviewer: craig.topper.
imkiva added a project: LLVM.
Herald added subscribers: jobnoorman, luke, VincentWu, vkmr, frasercrmck,
luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei,
PkmX, the_o, brucehoult, MartinMosbeck, rogfer0
imkiva created this revision.
imkiva added reviewers: clang, pcc.
imkiva added a project: clang.
Herald added a subscriber: cfe-commits.
Repository:
rC Clang
https://reviews.llvm.org/D71541
Files:
lib/AST/VTableBuilder.cpp
Index: lib/AST/VTableBuilder.cpp
==