[PATCH] D23667: [AAP] (3) Add AAP backend stub
Joerg Sonnenberger via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Dec 4 09:04:24 PST 2016
joerg added a comment.
Just being a bit pedantic here.
================
Comment at: lib/Target/AAP/CMakeLists.txt:5
+ AAPTargetMachine.cpp
+ )
+
----------------
Extra spaces.
================
Comment at: lib/Target/AAP/LLVMBuild.txt:1
+;===- ./lib/Target/AAP/LLVMBuild.txt -------------------------*- Conf -*--===;
+;
----------------
Missing - for line length.
================
Comment at: lib/Target/AAP/TargetInfo/AAPTargetInfo.cpp:27
+// Placeholder. This will be removed when the MC support has been
+// introduced
+extern "C" void LLVMInitializeAAPTargetMC() {
----------------
Missing period.
================
Comment at: lib/Target/AAP/TargetInfo/CMakeLists.txt:3
+ AAPTargetInfo.cpp
+ )
----------------
Extra space.
https://reviews.llvm.org/D23667
More information about the llvm-commits
mailing list