This revision was automatically updated to reflect the committed changes.
Closed by commit rG7aaa85627b59: [PowerPC] Add options to control paired vector
memops support (authored by bsaleil, committed by lei).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm
bsaleil updated this revision to Diff 281341.
bsaleil added a comment.
Update diff so it can be applied to master
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83722/new/
https://reviews.llvm.org/D83722
Files:
clang/include/clang/Driver/Options.
amyk accepted this revision as: amyk.
amyk added a comment.
This revision is now accepted and ready to land.
This LGTM. Thanks for addressing the comments Baptiste.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83722/new/
https://reviews.llvm.org/D
bsaleil marked an inline comment as done.
bsaleil added inline comments.
Comment at: llvm/lib/Target/PowerPC/PPC.td:243
+ "32Byte load and store instructions",
+ [FeatureISA3_0]>;
amyk wrote:
> Is this supposed to be `Feature
bsaleil updated this revision to Diff 279341.
bsaleil added a comment.
Add PairedVectorMemops to the list of unsupported features in a comment
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83722/new/
https://reviews.llvm.org/D83722
Files:
clang/
amyk added inline comments.
Comment at: llvm/lib/Target/PowerPC/PPC.td:243
+ "32Byte load and store instructions",
+ [FeatureISA3_0]>;
Is this supposed to be `FeatureISA3_1`?
Comment at: llvm/lib/Target/Pow
bsaleil updated this revision to Diff 277847.
bsaleil added a comment.
Add test to check `paired-vector-memops` is supported by the targets.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83722/new/
https://reviews.llvm.org/D83722
Files:
clang/in
bsaleil updated this revision to Diff 277838.
bsaleil added a comment.
Re-upload with surrounding context
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83722/new/
https://reviews.llvm.org/D83722
Files:
clang/include/clang/Driver/Options.td
cla
nemanjai added a comment.
Please re-upload this and provide the missing context.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83722/new/
https://reviews.llvm.org/D83722
___
cfe-commits mailing list
cf
bsaleil created this revision.
bsaleil added reviewers: PowerPC, power-llvm-team, nemanjai, lei, saghir, amyk,
stefanp.
bsaleil added projects: LLVM, clang, PowerPC.
Herald added subscribers: llvm-commits, cfe-commits, dang, steven.zhang,
shchenz, hiraditya.
This patch adds frontend and backend
10 matches
Mail list logo